From commits-return-40458-archive-asf-public=cust-asf.ponee.io@tinkerpop.apache.org Tue Sep 1 11:35:29 2020 Return-Path: X-Original-To: archive-asf-public@cust-asf.ponee.io Delivered-To: archive-asf-public@cust-asf.ponee.io Received: from mxout1-he-de.apache.org (mxout1-he-de.apache.org [95.216.194.37]) by mx-eu-01.ponee.io (Postfix) with ESMTPS id 94937180660 for ; Tue, 1 Sep 2020 13:35:29 +0200 (CEST) Received: from mail.apache.org (mailroute1-lw-us.apache.org [207.244.88.153]) by mxout1-he-de.apache.org (ASF Mail Server at mxout1-he-de.apache.org) with SMTP id E13B162CC1 for ; Tue, 1 Sep 2020 11:35:28 +0000 (UTC) Received: (qmail 77310 invoked by uid 500); 1 Sep 2020 11:35:28 -0000 Mailing-List: contact commits-help@tinkerpop.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@tinkerpop.apache.org Delivered-To: mailing list commits@tinkerpop.apache.org Received: (qmail 77301 invoked by uid 99); 1 Sep 2020 11:35:28 -0000 Received: from ec2-52-202-80-70.compute-1.amazonaws.com (HELO gitbox.apache.org) (52.202.80.70) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 01 Sep 2020 11:35:28 +0000 From: =?utf-8?q?GitBox?= To: commits@tinkerpop.apache.org Subject: =?utf-8?q?=5BGitHub=5D_=5Btinkerpop=5D_FlorianHockmann_commented_on_a_change?= =?utf-8?q?_in_pull_request_=231321=3A_TINKERPOP-2392_Improved_GLV_starting_?= =?utf-8?q?docs?= Message-ID: <159896012813.32230.9814311965592711717.asfpy@gitbox.apache.org> Date: Tue, 01 Sep 2020 11:35:28 -0000 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit In-Reply-To: References: FlorianHockmann commented on a change in pull request #1321: URL: https://github.com/apache/tinkerpop/pull/1321#discussion_r480962085 ########## File path: gremlin-dotnet/src/Gremlin.Net/Gremlin.Net.csproj ########## @@ -42,10 +42,20 @@ http://tinkerpop.apache.org/docs/current/reference/#gremlin-server or a remote graph provider that exposes protocols by which Gremlin.Net can connect. -Please see the reference documentation of Apache TinkerPop for more information on usage: https://s.apache.org/pgbwu +Please see the Reference Documentation of Apache TinkerPop for more information on usage: http://tinkerpop.apache.org/docs/current/reference Review comment: (nitpick) I think we can and should keep using `https` which also works for this link: https://tinkerpop.apache.org/docs/current/reference ########## File path: gremlin-dotnet/README.md ########## @@ -33,8 +33,22 @@ operating systems and with different .NET frameworks, such as .NET Framework and nuget install Gremlin.Net ``` -Please see the [reference documentation][docs] at Apache TinkerPop for more information. +The Gremlin language allows users to write highly expressive graph traversals and has a broad list of functions that +cover a wide body of features. The [Reference Documentation][steps] describes these functions and other aspects of the +TinkerPop ecosystem including some specifics on [Gremlin in .NET][docs] itself. Most of the examples found in the +documentation use Groovy language syntax in the [Gremlin Console][console]. For the most part, these examples +should generally translate to C# with [some logical modification][differences]. Given the strong correspondence +between canonical Gremlin in Java and its variants like C#, there is a limited amount of C#-specific +documentation and examples. This strong correspondence among variants ensures that the general Gremlin reference +documentation is applicable to all variants and that users moving between development languages can easily adopt the +Gremlin variant for that language. + +**NOTE** that versions suffixed with "-rc" are considered release candidates (i.e. pre-alpha, alpha, beta, etc.) and thus +for early testing purposes only. [tk]: http://tinkerpop.apache.org [gremlin]: http://tinkerpop.apache.org/gremlin.html -[docs]: http://tinkerpop.apache.org/docs/current/reference/#gremlin-DotNet \ No newline at end of file +[docs]: http://tinkerpop.apache.org/docs/current/reference/#gremlin-dotnet Review comment: Just a nit and not really related to this PR, but we can also upgrade these links to HTTPS now. ########## File path: docs/src/reference/gremlin-variants.asciidoc ########## @@ -1215,6 +1219,57 @@ and then it can be called from the application as follows: include::../../../gremlin-dotnet/test/Gremlin.Net.IntegrationTest/Docs/Reference/GremlinVariantsDslTests.cs[tags=dslExamples] ---- +[[gremlin-net-differences]] +=== Differences + +The biggest difference between Gremlin in .NET and the canonical version in Java is the casing of steps. Canonical +Gremlin utilizes `camelCase` as is typical in Java for function names, but C# utilizes `PascalCase` as it more typical Review comment: _utilizes `PascalCase` as it more typical_ should probably be _utilizes `PascalCase` as it **is** more typical_. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: users@infra.apache.org