You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@tinkerpop.apache.org by "dkuppitz (GitHub)" <gi...@apache.org> on 2018/12/03 21:49:14 UTC

[GitHub] [tinkerpop] dkuppitz commented on issue #1010: TINKERPOP-2075 Added ReferenceElementStrategy

Do you have a scenario in mind where providers would construct a new `ReferenceElement` step? I don't think it's necessary and I guess if you would have thought of such a scenario, you wouldn't have nested the class inside `ReferenceElementStrategy`; hence I think `ReferenceElement` constructor can be package-private.

Anyway, since this is just a minor thing ...

VOTE +1

[ Full content available at: https://github.com/apache/tinkerpop/pull/1010 ]
This message was relayed via gitbox.apache.org for dev@tinkerpop.apache.org