Original patch: "ZOOKEEPER-3598: Fix potential data inconsistency issue due to
has been reverted. Fangmin, my apologies for the inconvenience. Would you please create another PR with different solution or with Enrico’s workaround? Regards, Andor > On 2019. Nov 14., at 7:54, Andor Molnar <[email protected]> wrote: > > Hi Enrico, > > +1 for the idea > I think the best would be to do this before we cut the first version of 3.6.0. > > Btw I noticed that the trunk build is completely broken since > https://builds.apache.org/view/S-Z/view/ZooKeeper/job/ZooKeeper-trunk/769/ > > It’s failing on Java versions since we introduced those 2 commits. > But looks like the Maven build is still fine. > > Andor > > > >> On 2019. Nov 11., at 13:36, Enrico Olivelli <[email protected]> wrote: >> >> Hi community, >> I have just sent a PR to drop ANT from main build (not from >> recipes/contrib) and keep only Maven build. >> We are already releasing 3.5 branch with Maven, this is only the final step >> of this long story. >> Once we have only Maven it will be super easy to refactor the layout of the >> codebase (separate common/client/metrics provider apis...). >> >> https://github.com/apache/zookeeper/pull/1139 >> >> I really would like to commit this patch before cutting 3.6. >> btw we are not going to add other big refactors before 3.6, so it is not a >> blocker for the release. >> >> Enjoy >> >> Enrico >
