Il ven 15 nov 2019, 15:28 Andor Molnar <[email protected]> ha scritto:
> Hi, > > I noticed that PR#1139 has been merged. As a consequence I have to > remove/disable all of the Ant-based build jobs in this Jenkis dashboard: > https://builds.apache.org/view/S-Z/view/ZooKeeper/ > > Which also means that we don’t have a build job which is able to alert > when a test has crashed. > I can't follow you. We do have the maven jobs, don't we? Enrico > Is that okay? > > Andor > > > > > > > On 2019. Nov 15., at 11:06, Andor Molnar <[email protected]> wrote: > > > > 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 > >> > > > >
