mikepersonick commented on a change in pull request #1565:
URL: https://github.com/apache/tinkerpop/pull/1565#discussion_r804729876
##########
File path: gremlin-dotnet/src/Gremlin.Net/Process/Traversal/GraphTraversal.cs
##########
@@ -772,8 +772,16 @@ private GraphTraversal(ICollection<ITraversalStrategy>
traversalStrategies, Byte
/// </summary>
public GraphTraversal<S, E> HasId (object id, params object[] otherIds)
{
- var args = new List<object>(1 + otherIds.Length) {id};
Review comment:
Florian, thanks the review and merge on the other PR. I’m out today so
I’ll take a closer look on Monday, but I guess my general feeling is that
whatever the semantics are on `null` (or anything else), it’s probably best to
handle them in the query execution layer rather than the language bindings.
--
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]
