I think these changes make sense presuming that we don't register the same strategy more than once, which I don't think we do - @dkuppitz please think that one through a bit - is there any situation where we allow for that. Other than that, initializing the `ArrayList` in `flattenArguments()` and going to `LinkedHashSet` to avoid some looping seem to make sense to me.
All tests pass with `docker/build.sh -t -n -i` VOTE +1 [ Full content available at: https://github.com/apache/tinkerpop/pull/999 ] This message was relayed via gitbox.apache.org for [email protected]
