You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@zookeeper.apache.org by Enrico Olivelli <eo...@gmail.com> on 2023/06/16 10:47:38 UTC
Current master branch is broker
Hello ZooKeepers,
Current master branch is broken, see the errors below.
Maybe we committed patches and there were some unexpected conflicts
undetected by git.
Enrico
[ERROR] Failed to execute goal
org.apache.maven.plugins:maven-compiler-plugin:3.8.1:testCompile
(default-testCompile) on project zookeeper: Compilation failure:
Compilation failure:
[ERROR] /Users/enricoolivelli/dev/zookeeper/zookeeper-server/src/test/java/org/apache/zookeeper/test/PersistentRecursiveWatcherTest.java:[198,13]
no suitable method found for
assertEvent(java.util.concurrent.BlockingQueue<org.apache.zookeeper.WatchedEvent>,org.apache.zookeeper.Watcher.Event.EventType,java.lang.String)
[ERROR] method
org.apache.zookeeper.test.PersistentRecursiveWatcherTest.assertEvent(java.util.concurrent.BlockingQueue<org.apache.zookeeper.WatchedEvent>,org.apache.zookeeper.Watcher.Event.EventType,java.lang.String,org.apache.zookeeper.data.Stat)
is not applicable
[ERROR] (actual and formal argument lists differ in length)
[ERROR] method
org.apache.zookeeper.test.PersistentRecursiveWatcherTest.assertEvent(java.util.concurrent.BlockingQueue<org.apache.zookeeper.WatchedEvent>,org.apache.zookeeper.Watcher.Event.EventType,java.lang.String,long)
is not applicable
[ERROR] (actual and formal argument lists differ in length)
[ERROR] method
org.apache.zookeeper.test.PersistentRecursiveWatcherTest.assertEvent(java.util.concurrent.BlockingQueue<org.apache.zookeeper.WatchedEvent>,org.apache.zookeeper.Watcher.Event.EventType,org.apache.zookeeper.Watcher.Event.KeeperState,java.lang.String,long)
is not applicable
Re: Current master branch is broker
Posted by Kezhu Wang <ke...@gmail.com>.
Thank you Andor for managing this!
Best,
Kezhu Wang
On Fri, Jun 16, 2023 at 10:01 PM Andor Molnar <an...@apache.org> wrote:
>
> Fix submitted.
>
>
> On Fri, 2023-06-16 at 14:07 +0200, Andor Molnar wrote:
> > Yeah, sorry, I've submitted the patch which broke the build. My first
> > thought was to revert immediately and let the contributor to fix it,
> > but Kezhu already opened a PR with the fix.
> >
> > Once the build is greeen, I'm submitting it with the title:
> > ZOOKEEPER-4655: [ADDENDUM] fix build error
> >
> > Thanks Kezhu.
> >
> > Andor
> >
> >
> >
> > On Fri, 2023-06-16 at 19:44 +0800, tison wrote:
> > > Thank you Kezhu!
> > >
> > > This case somehow indicates that we're under a "high" traffic. lol.
> > >
> > > Best,
> > > tison.
> > >
> > >
> > > Kezhu Wang <ke...@gmail.com> 于2023年6月16日周五 19:20写道：
> > >
> > > > Hi Enrico,
> > > >
> > > > I presented a fix https://github.com/apache/zookeeper/pull/2012.
> > > >
> > > > On Fri, Jun 16, 2023 at 6:47 PM Enrico Olivelli <
> > > > eolivelli@gmail.com>
> > > > wrote:
> > > > > Hello ZooKeepers,
> > > > >
> > > > > Current master branch is broken, see the errors below.
> > > > >
> > > > > Maybe we committed patches and there were some unexpected
> > > > > conflicts
> > > > > undetected by git.
> > > > >
> > > > > Enrico
> > > > >
> > > > > [ERROR] Failed to execute goal
> > > > > org.apache.maven.plugins:maven-compiler-
> > > > > plugin:3.8.1:testCompile
> > > > > (default-testCompile) on project zookeeper: Compilation
> > > > > failure:
> > > > > Compilation failure:
> > > > >
> > > > > [ERROR]
> > > > /Users/enricoolivelli/dev/zookeeper/zookeeper-
> > > > server/src/test/java/org/apache/zookeeper/test/PersistentRecursiv
> > > > eW
> > > > atcherTest.java:[198,13]
> > > > > no suitable method found for
> > > > >
> > > > assertEvent(java.util.concurrent.BlockingQueue<org.apache.zookeep
> > > > er
> > > > .WatchedEvent>,org.apache.zookeeper.Watcher.Event.EventType,java.
> > > > la
> > > > ng.String)
> > > > > [ERROR] method
> > > > >
> > > > org.apache.zookeeper.test.PersistentRecursiveWatcherTest.assertEv
> > > > en
> > > > t(java.util.concurrent.BlockingQueue<org.apache.zookeeper.Watched
> > > > Ev
> > > > ent>,org.apache.zookeeper.Watcher.Event.EventType,java.lang.Strin
> > > > g,
> > > > org.apache.zookeeper.data.Stat)
> > > > > is not applicable
> > > > >
> > > > > [ERROR] (actual and formal argument lists differ in
> > > > > length)
> > > > >
> > > > > [ERROR] method
> > > > >
> > > > org.apache.zookeeper.test.PersistentRecursiveWatcherTest.assertEv
> > > > en
> > > > t(java.util.concurrent.BlockingQueue<org.apache.zookeeper.Watched
> > > > Ev
> > > > ent>,org.apache.zookeeper.Watcher.Event.EventType,java.lang.Strin
> > > > g,
> > > > long)
> > > > > is not applicable
> > > > >
> > > > > [ERROR] (actual and formal argument lists differ in
> > > > > length)
> > > > >
> > > > > [ERROR] method
> > > > >
> > > > org.apache.zookeeper.test.PersistentRecursiveWatcherTest.assertEv
> > > > en
> > > > t(java.util.concurrent.BlockingQueue<org.apache.zookeeper.Watched
> > > > Ev
> > > > ent>,org.apache.zookeeper.Watcher.Event.EventType,org.apache.zook
> > > > ee
> > > > per.Watcher.Event.KeeperState,java.lang.String,long)
> > > > > is not applicable
>
Re: Current master branch is broker
Posted by Andor Molnar <an...@apache.org>.
Fix submitted.
On Fri, 2023-06-16 at 14:07 +0200, Andor Molnar wrote:
> Yeah, sorry, I've submitted the patch which broke the build. My first
> thought was to revert immediately and let the contributor to fix it,
> but Kezhu already opened a PR with the fix.
>
> Once the build is greeen, I'm submitting it with the title:
> ZOOKEEPER-4655: [ADDENDUM] fix build error
>
> Thanks Kezhu.
>
> Andor
>
>
>
> On Fri, 2023-06-16 at 19:44 +0800, tison wrote:
> > Thank you Kezhu!
> >
> > This case somehow indicates that we're under a "high" traffic. lol.
> >
> > Best,
> > tison.
> >
> >
> > Kezhu Wang <ke...@gmail.com> 于2023年6月16日周五 19:20写道：
> >
> > > Hi Enrico,
> > >
> > > I presented a fix https://github.com/apache/zookeeper/pull/2012.
> > >
> > > On Fri, Jun 16, 2023 at 6:47 PM Enrico Olivelli <
> > > eolivelli@gmail.com>
> > > wrote:
> > > > Hello ZooKeepers,
> > > >
> > > > Current master branch is broken, see the errors below.
> > > >
> > > > Maybe we committed patches and there were some unexpected
> > > > conflicts
> > > > undetected by git.
> > > >
> > > > Enrico
> > > >
> > > > [ERROR] Failed to execute goal
> > > > org.apache.maven.plugins:maven-compiler-
> > > > plugin:3.8.1:testCompile
> > > > (default-testCompile) on project zookeeper: Compilation
> > > > failure:
> > > > Compilation failure:
> > > >
> > > > [ERROR]
> > > /Users/enricoolivelli/dev/zookeeper/zookeeper-
> > > server/src/test/java/org/apache/zookeeper/test/PersistentRecursiv
> > > eW
> > > atcherTest.java:[198,13]
> > > > no suitable method found for
> > > >
> > > assertEvent(java.util.concurrent.BlockingQueue<org.apache.zookeep
> > > er
> > > .WatchedEvent>,org.apache.zookeeper.Watcher.Event.EventType,java.
> > > la
> > > ng.String)
> > > > [ERROR] method
> > > >
> > > org.apache.zookeeper.test.PersistentRecursiveWatcherTest.assertEv
> > > en
> > > t(java.util.concurrent.BlockingQueue<org.apache.zookeeper.Watched
> > > Ev
> > > ent>,org.apache.zookeeper.Watcher.Event.EventType,java.lang.Strin
> > > g,
> > > org.apache.zookeeper.data.Stat)
> > > > is not applicable
> > > >
> > > > [ERROR] (actual and formal argument lists differ in
> > > > length)
> > > >
> > > > [ERROR] method
> > > >
> > > org.apache.zookeeper.test.PersistentRecursiveWatcherTest.assertEv
> > > en
> > > t(java.util.concurrent.BlockingQueue<org.apache.zookeeper.Watched
> > > Ev
> > > ent>,org.apache.zookeeper.Watcher.Event.EventType,java.lang.Strin
> > > g,
> > > long)
> > > > is not applicable
> > > >
> > > > [ERROR] (actual and formal argument lists differ in
> > > > length)
> > > >
> > > > [ERROR] method
> > > >
> > > org.apache.zookeeper.test.PersistentRecursiveWatcherTest.assertEv
> > > en
> > > t(java.util.concurrent.BlockingQueue<org.apache.zookeeper.Watched
> > > Ev
> > > ent>,org.apache.zookeeper.Watcher.Event.EventType,org.apache.zook
> > > ee
> > > per.Watcher.Event.KeeperState,java.lang.String,long)
> > > > is not applicable
Re: Current master branch is broker
Posted by Andor Molnar <an...@apache.org>.
Yeah, sorry, I've submitted the patch which broke the build. My first
thought was to revert immediately and let the contributor to fix it,
but Kezhu already opened a PR with the fix.
Once the build is greeen, I'm submitting it with the title:
ZOOKEEPER-4655: [ADDENDUM] fix build error
Thanks Kezhu.
Andor
On Fri, 2023-06-16 at 19:44 +0800, tison wrote:
> Thank you Kezhu!
>
> This case somehow indicates that we're under a "high" traffic. lol.
>
> Best,
> tison.
>
>
> Kezhu Wang <ke...@gmail.com> 于2023年6月16日周五 19:20写道：
>
> > Hi Enrico,
> >
> > I presented a fix https://github.com/apache/zookeeper/pull/2012.
> >
> > On Fri, Jun 16, 2023 at 6:47 PM Enrico Olivelli <
> > eolivelli@gmail.com>
> > wrote:
> > > Hello ZooKeepers,
> > >
> > > Current master branch is broken, see the errors below.
> > >
> > > Maybe we committed patches and there were some unexpected
> > > conflicts
> > > undetected by git.
> > >
> > > Enrico
> > >
> > > [ERROR] Failed to execute goal
> > > org.apache.maven.plugins:maven-compiler-plugin:3.8.1:testCompile
> > > (default-testCompile) on project zookeeper: Compilation failure:
> > > Compilation failure:
> > >
> > > [ERROR]
> > /Users/enricoolivelli/dev/zookeeper/zookeeper-
> > server/src/test/java/org/apache/zookeeper/test/PersistentRecursiveW
> > atcherTest.java:[198,13]
> > > no suitable method found for
> > >
> > assertEvent(java.util.concurrent.BlockingQueue<org.apache.zookeeper
> > .WatchedEvent>,org.apache.zookeeper.Watcher.Event.EventType,java.la
> > ng.String)
> > > [ERROR] method
> > >
> > org.apache.zookeeper.test.PersistentRecursiveWatcherTest.assertEven
> > t(java.util.concurrent.BlockingQueue<org.apache.zookeeper.WatchedEv
> > ent>,org.apache.zookeeper.Watcher.Event.EventType,java.lang.String,
> > org.apache.zookeeper.data.Stat)
> > > is not applicable
> > >
> > > [ERROR] (actual and formal argument lists differ in length)
> > >
> > > [ERROR] method
> > >
> > org.apache.zookeeper.test.PersistentRecursiveWatcherTest.assertEven
> > t(java.util.concurrent.BlockingQueue<org.apache.zookeeper.WatchedEv
> > ent>,org.apache.zookeeper.Watcher.Event.EventType,java.lang.String,
> > long)
> > > is not applicable
> > >
> > > [ERROR] (actual and formal argument lists differ in length)
> > >
> > > [ERROR] method
> > >
> > org.apache.zookeeper.test.PersistentRecursiveWatcherTest.assertEven
> > t(java.util.concurrent.BlockingQueue<org.apache.zookeeper.WatchedEv
> > ent>,org.apache.zookeeper.Watcher.Event.EventType,org.apache.zookee
> > per.Watcher.Event.KeeperState,java.lang.String,long)
> > > is not applicable
Re: Current master branch is broker
Posted by tison <wa...@gmail.com>.
Thank you Kezhu!
This case somehow indicates that we're under a "high" traffic. lol.
Best,
tison.
Kezhu Wang <ke...@gmail.com> 于2023年6月16日周五 19:20写道：
> Hi Enrico,
>
> I presented a fix https://github.com/apache/zookeeper/pull/2012.
>
> On Fri, Jun 16, 2023 at 6:47 PM Enrico Olivelli <eo...@gmail.com>
> wrote:
> >
> > Hello ZooKeepers,
> >
> > Current master branch is broken, see the errors below.
> >
> > Maybe we committed patches and there were some unexpected conflicts
> > undetected by git.
> >
> > Enrico
> >
> > [ERROR] Failed to execute goal
> > org.apache.maven.plugins:maven-compiler-plugin:3.8.1:testCompile
> > (default-testCompile) on project zookeeper: Compilation failure:
> > Compilation failure:
> >
> > [ERROR]
> /Users/enricoolivelli/dev/zookeeper/zookeeper-server/src/test/java/org/apache/zookeeper/test/PersistentRecursiveWatcherTest.java:[198,13]
> > no suitable method found for
> >
> assertEvent(java.util.concurrent.BlockingQueue<org.apache.zookeeper.WatchedEvent>,org.apache.zookeeper.Watcher.Event.EventType,java.lang.String)
> >
> > [ERROR] method
> >
> org.apache.zookeeper.test.PersistentRecursiveWatcherTest.assertEvent(java.util.concurrent.BlockingQueue<org.apache.zookeeper.WatchedEvent>,org.apache.zookeeper.Watcher.Event.EventType,java.lang.String,org.apache.zookeeper.data.Stat)
> > is not applicable
> >
> > [ERROR] (actual and formal argument lists differ in length)
> >
> > [ERROR] method
> >
> org.apache.zookeeper.test.PersistentRecursiveWatcherTest.assertEvent(java.util.concurrent.BlockingQueue<org.apache.zookeeper.WatchedEvent>,org.apache.zookeeper.Watcher.Event.EventType,java.lang.String,long)
> > is not applicable
> >
> > [ERROR] (actual and formal argument lists differ in length)
> >
> > [ERROR] method
> >
> org.apache.zookeeper.test.PersistentRecursiveWatcherTest.assertEvent(java.util.concurrent.BlockingQueue<org.apache.zookeeper.WatchedEvent>,org.apache.zookeeper.Watcher.Event.EventType,org.apache.zookeeper.Watcher.Event.KeeperState,java.lang.String,long)
> > is not applicable
>
Re: Current master branch is broker
Posted by Kezhu Wang <ke...@gmail.com>.
Hi Enrico,
I presented a fix https://github.com/apache/zookeeper/pull/2012.
On Fri, Jun 16, 2023 at 6:47 PM Enrico Olivelli <eo...@gmail.com> wrote:
>
> Hello ZooKeepers,
>
> Current master branch is broken, see the errors below.
>
> Maybe we committed patches and there were some unexpected conflicts
> undetected by git.
>
> Enrico
>
> [ERROR] Failed to execute goal
> org.apache.maven.plugins:maven-compiler-plugin:3.8.1:testCompile
> (default-testCompile) on project zookeeper: Compilation failure:
> Compilation failure:
>
> [ERROR] /Users/enricoolivelli/dev/zookeeper/zookeeper-server/src/test/java/org/apache/zookeeper/test/PersistentRecursiveWatcherTest.java:[198,13]
> no suitable method found for
> assertEvent(java.util.concurrent.BlockingQueue<org.apache.zookeeper.WatchedEvent>,org.apache.zookeeper.Watcher.Event.EventType,java.lang.String)
>
> [ERROR] method
> org.apache.zookeeper.test.PersistentRecursiveWatcherTest.assertEvent(java.util.concurrent.BlockingQueue<org.apache.zookeeper.WatchedEvent>,org.apache.zookeeper.Watcher.Event.EventType,java.lang.String,org.apache.zookeeper.data.Stat)
> is not applicable
>
> [ERROR] (actual and formal argument lists differ in length)
>
> [ERROR] method
> org.apache.zookeeper.test.PersistentRecursiveWatcherTest.assertEvent(java.util.concurrent.BlockingQueue<org.apache.zookeeper.WatchedEvent>,org.apache.zookeeper.Watcher.Event.EventType,java.lang.String,long)
> is not applicable
>
> [ERROR] (actual and formal argument lists differ in length)
>
> [ERROR] method
> org.apache.zookeeper.test.PersistentRecursiveWatcherTest.assertEvent(java.util.concurrent.BlockingQueue<org.apache.zookeeper.WatchedEvent>,org.apache.zookeeper.Watcher.Event.EventType,org.apache.zookeeper.Watcher.Event.KeeperState,java.lang.String,long)
> is not applicable