[GitHub] [tinkerpop] spmallette commented on pull request #1699: fix: Traversal.graph is empty in StepStrategy.apply() with `count().is(0)`

["spmallette (via GitHub)" <gi...@apache.org>]

spmallette commented on PR #1699:
URL: https://github.com/apache/tinkerpop/pull/1699#issuecomment-1513638447

   @javeme this is a bit old now, but it seems we must revisit it. the fix mentioned here is being rolled back for 3.5.6/3.6.3. that extra recursion is proving too costly. please see the latest work on this here: https://github.com/apache/tinkerpop/pull/2026 if you find the `Graph` still isn't there for some reason, you can pull it with `Traversal.getRoot(traversal).getGraph()` or in the worst case, we might want to look into figuring out where else we might get the `Graph` assigned as part of traversal construction, but so far, five graph implementations seem ok with how this change works.


-- 
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: commits-unsubscribe@tinkerpop.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org