The significant change is ffe1b4c Marko A. Rodriguez <okramma...@gmail.com> on 2016/11/15 at 12:44 AM
The description is "added TraversalHelper.addHasContainer() which will either append a HasStep with container or if the traverasl ends with a HasContainerHolder, fold the container into the holder. This just makes the code in GraphTravesrsal cleaner with less copy/paste." I don't have a problem with the changes just that is quite a refactor on my side as it changes the structure of the HasSteps and its HasContainers. I had made some assumptions around how HasSteps and HasContainers look when optimizing. The change is quite old but somehow it was not present in the 3.2.4-SNAPSHOT I tested on it when the code freeze announcement was made. Not sure what happened there but alas its not giving me enough time to get things working again. So I am not voting negative just requesting a weekend, if possible, to get through the refactor. Cheers Pieter "added TraversalHelper.addHasContainer() on 2016/11/15 On 09/02/2017 22:11, Stephen Mallette wrote: > Unless I'm missing something, HasStep hasn't changed in 4 months and > HasContainer hasn't changed in 3 months. The only update that went in after > 2/2 that I can think of that would have any bearing for graph providers who > tested before/after that date would be the AutoCloseable stuff: > > https://github.com/apache/tinkerpop/pull/548 > > Your problems aren't related to that are they? Can you provide some > synopsis of where the problems lie? > > > > On Thu, Feb 9, 2017 at 2:47 PM, pieter-gmail <pieter.mar...@gmail.com> > wrote: > >> Hi, >> >> Some issues regarding the release process. >> >> On the 2/2/2017 the 3.2.4-SNAPSHOT was released. >> I then started testing and found almost no issues. >> >> However yesterday when the VOTE mail came I found many issues on 3.2.4. >> To understand the confusion I tested again on 3.2.4-SNAPSHOT and found >> the same new issues. >> I then checked the 3.2.4-SNAPSHOT timestamp and it changed to 08/02/2017 >> >> Not sure what happened there as I can not, nor is it worth it, check an >> old SNAPSHOT version's binary. >> The relevant code changes are quite old (December 2016) but it may have >> been done on a separate branch and rebasing may loose the merging >> information. Not sure about this though. >> >> This only leaves the 72 hours to catch up. >> >> Even though it is a minor release there has been significant changes to >> HasStep and HasContainer which breaks the heart (mine at least) of >> implementors optimization code. >> >> Basically to get any value from a vote I for one will need more time. >> >> So far I have not found any TinkerPop issues but will need at least the >> weekend to know better. >> >> Thanks >> Pieter >> >> On 08/02/2017 16:51, Jason Plurad wrote: >>> Hello, >>> >>> We are happy to announce that TinkerPop 3.2.4 is ready for release. >>> >>> The release artifacts can be found at this location: >>> https://dist.apache.org/repos/dist/dev/tinkerpop/3.2.4/ >>> >>> The source distribution is provided by: >>> apache-tinkerpop-3.2.4-src.zip >>> >>> Two binary distributions are provided for user convenience: >>> apache-tinkerpop-gremlin-console-3.2.4-bin.zip >>> apache-tinkerpop-gremlin-server-3.2.4-bin.zip >>> >>> The GPG key used to sign the release artifacts is available at: >>> https://dist.apache.org/repos/dist/dev/tinkerpop/KEYS >>> >>> The online docs can be found here: >>> http://tinkerpop.apache.org/docs/3.2.4/ (user docs) >>> http://tinkerpop.apache.org/docs/3.2.4/upgrade/ (upgrade docs) >>> http://tinkerpop.apache.org/javadocs/3.2.4/core/ (core javadoc) >>> http://tinkerpop.apache.org/javadocs/3.2.4/full/ (full javadoc) >>> >>> The tag in Apache Git can be found here: >>> >>> https://git-wip-us.apache.org/repos/asf?p=tinkerpop.git;a= >> tag;h=refs/tags/3.2.4 >>> The release notes are available here: >>> >>> https://github.com/apache/tinkerpop/blob/3.2.4/ >> CHANGELOG.asciidoc#release-3-2-4 >>> The [VOTE] will be open for the next 72 hours --- closing Saturday >>> (February 11, 2017) at 10:00 AM EST. >>> >>> My vote is +1. >>> >>> Thank you very much, >>> Jason Plurad >>> >>