You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@geode.apache.org by "ASF subversion and git services (Jira)" <ji...@apache.org> on 2021/12/11 02:32:00 UTC

[jira] [Commented] (GEODE-7282) Geode Native Client .NET guide: Fix capitalization issues

    [ https://issues.apache.org/jira/browse/GEODE-7282?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17457513#comment-17457513 ] 

ASF subversion and git services commented on GEODE-7282:
--------------------------------------------------------

Commit 0df232884bd3025fde5a0f948b30072874419e20 in geode-native's branch refs/heads/master from Dave Barnes
[ https://gitbox.apache.org/repos/asf?p=geode-native.git;h=0df2328 ]

GEODE-7282: Geode Native Client .NET guide: Fix capitalization issues (#863)



> Geode Native Client .NET guide: Fix capitalization issues
> ---------------------------------------------------------
>
>                 Key: GEODE-7282
>                 URL: https://issues.apache.org/jira/browse/GEODE-7282
>             Project: Geode
>          Issue Type: Bug
>          Components: docs, native client
>            Reporter: Dave Barnes
>            Assignee: Dave Barnes
>            Priority: Major
>              Labels: pull-request-available
>             Fix For: 1.12.5, 1.13.5, 1.14.1, 1.15.0
>
>
> Reviewer [~karensmolermiller] approved the Native Client manual reorganization (GEODE-7262), but noted the following issues with the .NET version. These need cleaning up.
> geode-native/dotnet/111/regions/registering-interest-for-entries.html
> - C++ capitalization in 3rd paragraph and in Client API for Registering Interest
> subsection
> - I believe that all the examples use C++ capitalization
>  
> geode-native/dotnet/111/serialization/data-serialization.html
> - The .NET Example subsection has capitalization wrong for the field name
> description. I think that the example was wrong before these docs were
> separated, so this is not a mistake in separation.
>  
> geode-native/dotnet/111/remote-queries.html
> - Step 3 under Executing a Query from the Client should have a capital 'E'
> in Query.Execute(). I think that the example was wrong before these docs were
> separated, so this is not a mistake in separation.
>  
> geode-native/dotnet/111/continuous-queries.html
> - Prose within step 2 of Typical Continuous Query Lifecycle should change
> to CqQuery Execute* methods (in 2 spots).  Original prose only used the
> C++ capitalization.  Step 4 changes to CqQuery.Close.
> - Prose within step 3 of Executing a Continuous Query from the Client
> should change to CqQuery Execute* methods.
>  
> geode-native/dotnet/111/function-execution.html
> - This whole section was originally written with C++ capitalization. I suggest
> that we introduce a separate task/PR to correct the capitalization for the
> .NET docs.
>  
> geode-native/dotnet/111/transactions.html
> - first paragraph of Native Client Transaction APIs should capitalize Begin,
> Commit, and Rollback, as they are function names.
> - second paragraph of Running Native Client Transaction should capitalize Begin,
> Commit, and Rollback, as they are function names.
> - There's a boo-boo in capitalization of the .NET Example. Within a foreach
> code block, it should be
>     var value = GetValueFromExternalSystem();
> because function name start with a capital letter.



--
This message was sent by Atlassian Jira
(v8.20.1#820001)