Github user okram commented on the issue:
https://github.com/apache/tinkerpop/pull/447
If you guys don't mind, I'm going to change the method signatures (the NEW
methods added -- nothing from before changed).
{code}
withStrategy(String, Object...) -> withStrategies(Map<String,Object>...)
withoutStrategy(String) -> withoutStrategies(String...)
{code}
The reason this is smart is:
1. Registering strategies is expensive as it costs a `TraversalSource`
clone and a `TraversalStrategies` sort. This is why doing all strategy
declarations in one method is smart.
2. While creating `Map`s in Java is horrendous, Java users would NOT use
these methods. They would use the Java object version of these methods.
However, for Gremlin-Python, Groovy, JavaScript -- creating maps is natural and
looks good. Also, we don't run into the "multi-key" issues.
Nothing fudamental here, just some asethetics. If you don't object, I will
just do this, update docs, and merge.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
