spmallette commented on code in PR #3212: URL: https://github.com/apache/tinkerpop/pull/3212#discussion_r2362544813
########## 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: please include examples. what patterns will users find themselves needing to look at when they upgrade? what are the performance impacts? how do they mitigate? -- 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]
