spmallette commented on code in PR #1808:
URL: https://github.com/apache/tinkerpop/pull/1808#discussion_r970742051
##########
gremlin-dotnet/src/Gremlin.Net/Process/Traversal/GraphTraversal.cs:
##########
@@ -141,73 +159,73 @@ private GraphTraversal(ICollection<ITraversalStrategy>
traversalStrategies, Byte
/// <summary>
/// Adds the aggregate step to this <see
cref="GraphTraversal{SType, EType}" />.
/// </summary>
- public GraphTraversal<S, E> Aggregate (Scope scope, string
sideEffectKey)
+ public GraphTraversal<S, E_> Aggregate (Scope scope, string
sideEffectKey)
{
Bytecode.AddStep("aggregate", scope, sideEffectKey);
- return Wrap<S, E>(this);
+ return Wrap<S, E_>(this);
Review Comment:
@vkagamlyk would you mind taking the extra step to fix it on this PR? or an
immediate follow-up PR? i'd hate for us to forget this change when we have a
chance to instituted it on 3.7.x and let this release with the underscore.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
