Thanks Fangmin for pointing out these issues. I’ll take a look at them once I have some free cycles - currently on holiday. :)
We should consider compiling a list of 3.6 blockers in Jira similar what we had for 3.5. Andor > On 2019. Jun 26., at 1:26, Fangmin Lv <lvfang...@gmail.com> wrote: > > It's great to have a 3.6.0 release, currently all the FB contributed > features has been running inside FB for more than a month, so it > should be stable enough for community to use. > > Also I agreed with Patrick's point to review all flags and consider to turn > on by default. > > For the pending PRs, the following might be higher priority and would be > nice to include in the 3.6.0 release: > > * ZOOKEEPER-3356: Implement advanced Netty flow control based on feedback > from ZK to avoid OOM issue > * ZOOKEEPER-3145: Avoid watch missing issue due to stale pzxid when > replaying CloseSession txn with fuzzy snapshot > * ZOOKEEPER-3240: Close socket on Learner shutdown to avoid dangling socket > > Thanks, > Fangmin > > On Sat, Jun 15, 2019 at 9:21 AM Patrick Hunt <ph...@apache.org> wrote: > >> Good idea. Agree on including anything we've postponed to a new cycle - the >> patch from mapr is an obvious one to consider. >> >> We should also look at things we've disabled by default and consider >> whether we can turn them on by default. If not why not, and what can we do >> to fix this in a subsequent release. >> >> Have we deprecated anything that we should now remove? >> >> Also a good time to review the state of Java versions and make changes wrt >> supported versions and so forth. >> >> There was a proposal to remove contribs, or at least consider the ones that >> are still valuable vs moving some out. We should do that as well. >> >> Regards, >> >> Patrick >> >> On Sat, Jun 15, 2019 at 9:02 AM Jordan Zimmerman < >> jor...@jordanzimmerman.com> >> wrote: >> >>> On Persistent/Recursive watches: I’m willing to rebase, etc if there’s >>> confidence it will be merged. >>> >>> ==================== >>> Jordan Zimmerman >>> >>>> On Jun 15, 2019, at 10:59 AM, Andor Molnar <an...@cloudera.com.invalid >>> >>> wrote: >>>> >>>> Hi Enrico! >>>> >>>> Very good point, I entirely support the idea. >>>> >>>> Question to Friends@Facebook and Twitter contributors: how many >>> outstanding >>>> Jiras/PRs do you have which you would like to see in 3.6? >>>> >>>> I'd also like to highlight the long outstanding PR from Mapr: >>>> https://github.com/apache/zookeeper/pull/730 >>>> >>>> And some great new features which are still looking for to be merged: >>>> - Persistent recursive watchers: >>>> https://github.com/apache/zookeeper/pull/136 >>>> - Enforce client auth: https://github.com/apache/zookeeper/pull/118 >>>> - Slow operation log >>>> - Jetty port unification >>>> >>>> Regards, >>>> Andor >>>> >>>> >>>> >>>> >>>> >>>>> On Sat, Jun 15, 2019 at 1:31 PM Enrico Olivelli <eolive...@gmail.com> >>> wrote: >>>>> >>>>> Hi Zookeepers ! >>>>> I checked on JIRA and it seems that master in good shape, no real >>> blockers >>>>> that mine the stability of the code. >>>>> >>>>> We have plenty of cool pull requests almost ready to be merged (mostly >>> from >>>>> Facebook friends and Twitter fork) >>>>> >>>>> Current master branch is full of great features in respect to 3.5. >>>>> >>>>> AFAIK There is no incompatibility with 3.5 so it is okay to stay with >>>>> 3.6.0, although I think that there is so much stuff to legit a switch >> to >>>>> 4.0.0 (but we can reserve such bump for the time we will separate the >>> java >>>>> client and create a minimal compatibility breakage) >>>>> >>>>> Thoughts? >>>>> >>>>> Enrico >>>>> >>> >>