spmallette commented on code in PR #3212: URL: https://github.com/apache/tinkerpop/pull/3212#discussion_r2375548193
########## docs/src/upgrade/release-3.8.x.asciidoc: ########## @@ -699,6 +699,17 @@ some of the interfaces in the gremlin traversal engine. This has led to several As of 3.8.0 `with()` modulation of the following steps will no longer work: `addV()`, `addE()`, `property()`, `drop()`, `mergeV()`, and `mergeE()`. +==== Stricter RepeatUnrollStrategy + +The `RepeatUnrollStrategy` has been updated to use a more conservative approach for determining which repeat traversals +are safe to unroll. Previously, the strategy would attempt to unroll most repeat loops. This caused unintentional +traversal semantic changes when some steps were unrolled (especially barrier steps). Review Comment: what you added helps. maybe something still missing is that `RepeatUnrollStrategy` really only applies when using `times` right? i think that would help narrow down the patterns that users would look for if that were made more clear in the introductory paragraph and perhaps down in the migration section. -- 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]
