[GitHub] tinkerpop issue #811: TINKERPOP-1758 Apply RemoteStrategy before all Decorat...
Github user spmallette commented on the issue: https://github.com/apache/tinkerpop/pull/811 fair enough - i kinda wanted to just fix this problem since it was out there for so long. i'm hesitant to just hardcode the `RemoteStrategy` as "first" - something feels off about doing that. perhaps we wait until this approach presents as an issue before building lots of new infrastructure to support something more flexible. ---
[GitHub] tinkerpop issue #811: TINKERPOP-1758 Apply RemoteStrategy before all Decorat...
Github user robertdale commented on the issue: https://github.com/apache/tinkerpop/pull/811 I don't know. Just asking for the record. VOTE +1 ---
[GitHub] tinkerpop issue #811: TINKERPOP-1758 Apply RemoteStrategy before all Decorat...
Github user spmallette commented on the issue: https://github.com/apache/tinkerpop/pull/811 no - this only works by convention @robertdale - i don't think there's a way to do that short of scanning the classpath somehow for "decorations" and ordering them properly. or perhaps something more drastic that mucked with strategy ordering that always placed `RemoteStrategy` first, but i was hesitant to mess with that too much. i dunno - am i not thinking of another way to implement this? ---
[GitHub] tinkerpop issue #811: TINKERPOP-1758 Apply RemoteStrategy before all Decorat...
Github user robertdale commented on the issue: https://github.com/apache/tinkerpop/pull/811 Does this handle custom strategies? ---
[GitHub] tinkerpop issue #811: TINKERPOP-1758 Apply RemoteStrategy before all Decorat...
Github user dkuppitz commented on the issue: https://github.com/apache/tinkerpop/pull/811 VOTE: +1 ---
