Cole-Greer commented on code in PR #3254:
URL: https://github.com/apache/tinkerpop/pull/3254#discussion_r2464420735
##########
gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/sideEffect/GroupCountTest.java:
##########
@@ -237,6 +238,7 @@ public void
g_V_unionXoutXknowsX__outXcreatedX_inXcreatedXX_groupCount_selectXva
@Test
@LoadGraphWith(MODERN)
+ @Ignore("There is a bug in OLAP which prevents LocalBarrier steps from
following cap() in a traversal.")
Review Comment:
Note: I seem to have stumbled into a longstanding bug in OLAP with this
change, as this `LocalBarrier->Cap->LocalBarrier->Cap` traversal structure
results in workers setting global side effects during execution which is not
permitted. This sort of query structure previously failed with AggregateGlobal,
and is now failing with GroupCount as well due to the addition of LocalBarrier.
```
g.V().both().aggregate("a").out().cap("a").unfold().both().aggregate("b").cap("b")
java.lang.RuntimeException: java.lang.IllegalStateException:
java.lang.IllegalArgumentException: The memory can only be set() during vertex
program setup and terminate: a
```
I'm currently considering a followup PR to resolve this bug.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
