The problem with the oom is that the test case doesn't complete. I had to change the severity level from INFO to WARN to get it to run. For the other problem, I don't have enough insight yet.
-Flavio > -----Original Message----- > From: Ivan Kelly [mailto:iv...@apache.org] > Sent: 13 September 2013 10:50 > To: bookkeeper-...@zookeeper.apache.org > Cc: bookkeeper-u...@zookeeper.apache.org > Subject: Re: [VOTE] Bookkeeper 4.2.2 release candidate 0 > > The oom issue isn't a blocker (the problem is zk's oververbose logging). > BOOKKEEPER-683 is though. > > I agree we should cancel this rc. However, it will probably be october before I > can cut a new one. Busy couple of weeks coming up. > > -Ivan > > > On Thu, Sep 12, 2013 at 09:35:39PM +0100, Flavio Junqueira wrote: > > I'm getting the same problem with GcLedgersTest: > > > > Exception in thread "ThreadedStreamConsumer" > java.lang.OutOfMemoryError: Java heap space > > at java.util.Arrays.copyOf(Arrays.java:2367) > > at > java.lang.AbstractStringBuilder.expandCapacity(AbstractStringBuilder.java:13 > 0) > > at > java.lang.AbstractStringBuilder.ensureCapacityInternal(AbstractStringBuilder > .java:114) > > at > java.lang.AbstractStringBuilder.append(AbstractStringBuilder.java:415) > > at java.lang.StringBuffer.append(StringBuffer.java:237) > > at > org.apache.maven.surefire.report.ConsoleOutputFileReporter.writeMessag > e(ConsoleOutputFileReporter.java:115) > > at > org.apache.maven.surefire.report.MulticastingReporter.writeMessage(Multi > castingReporter.java:101) > > at > org.apache.maven.surefire.report.TestSetRunListener.writeTestOutput(Test > SetRunListener.java:99) > > at > org.apache.maven.plugin.surefire.booterclient.output.ForkClient.consumeLi > ne(ForkClient.java:132) > > at > org.apache.maven.plugin.surefire.booterclient.output.ThreadedStreamCons > umer$Pumper.run(ThreadedStreamConsumer.java:67) > > at java.lang.Thread.run(Thread.java:722) > > > > and one test failure on hedwig-server (BOOKKEEPER-683). I don't want to > be a party pooper, but I'd feel more comfortable if we could fix these issues > and have another release candidate. What do you think? > > > > -Flavio > > > > > > On Sep 12, 2013, at 3:27 PM, Jiannan Wang <jian...@yahoo-inc.com> > wrote: > > > > > Agree, thanks for the finding. > > > > > > Best, > > > Jiannan > > > > > > On 9/12/13 6:05 PM, "Ivan Kelly" <iv...@apache.org> wrote: > > > > > >> I managed to repro with > > >> MAVEN_OPTS="-Xmx128M" mvn test -Dtest=GcLedgersTest > > >> > > >> The test in question creates 30000 ledgers, using > > >> asyncCreateLedgers, which causes a lot of zk INFO level log messages. > > >> > > >> This isn't an issue worth blocking the release for. > > >> > > >> -Ivan > > >> > > >> On Thu, Sep 12, 2013 at 09:42:49AM +0000, Ivan Kelly wrote: > > >>> Hi Jiannan, > > >>> > > >>> Could you send me the logs for this. The <argLine> in the pom has > > >>> no affect here because that only applies to the surefire child > > >>> process. This OOM is in the maven process itself, because the test > > >>> is producing too many logs. I'll try reducing my heap to see if I > > >>> can get the same to happen. > > >>> > > >>> -Ivan > > >>> > > >>> > > >>> On Thu, Sep 12, 2013 at 03:31:19AM +0000, Jiannan Wang wrote: > > >>>> +1 for this release. > > >>>> > > >>>> All test cases pass except > > >>>> org.apache.bookkeeper.meta.GcLedgersTest, > > >>> which > > >>>> throws following exception > > >>>> ---------- > > >>>> Exception in thread "ThreadedStreamConsumer" > > >>> java.lang.OutOfMemoryError: > > >>>> Java heap space > > >>>> at java.util.Arrays.copyOf(Arrays.java:2882) > > >>>> at > > >>>> > > >>> java.lang.AbstractStringBuilder.expandCapacity(AbstractStringBuild > > >>> er.java > > >>> :1 > > >>>> 00) > > >>>> at > > >>> java.lang.AbstractStringBuilder.append(AbstractStringBuilder.java: > > >>> 390) > > >>>> at java.lang.StringBuffer.append(StringBuffer.java:224) > > >>>> at > > >>>> > > >>> > org.apache.maven.surefire.report.ConsoleOutputFileReporter.writeMe > > >>> ssage(C > > >>> on > > >>>> soleOutputFileReporter.java:115) > > >>>> at > > >>>> > > >>> > org.apache.maven.surefire.report.MulticastingReporter.writeMessage > > >>> (Multic > > >>> as > > >>>> tingReporter.java:101) > > >>>> at > > >>>> > > >>> > org.apache.maven.surefire.report.TestSetRunListener.writeTestOutpu > > >>> t(TestS > > >>> et > > >>>> RunListener.java:99) > > >>>> at > > >>>> > > >>> org.apache.maven.plugin.surefire.booterclient.output.ForkClient.co > > >>> nsumeLi > > >>> ne > > >>>> (ForkClient.java:132) > > >>>> at > > >>>> > > >>> org.apache.maven.plugin.surefire.booterclient.output.ThreadedStrea > > >>> mConsum > > >>> er > > >>>> $Pumper.run(ThreadedStreamConsumer.java:67) > > >>>> at java.lang.Thread.run(Thread.java:680) > > >>>> ---------- > > >>>> > > >>>> I need to execute 'export MAVEN_OPTS="-Xmx1G"' before running > the > > >>> test. > > >>>> (So the <argLine>-Xmx1G -Djava.net.preferIPv4Stack=true</argLine> > > >>>> in pom.xml takes no effect? My maven version is Apache Maven > > >>>> 3.0.3 > > >>> (r1075438; > > >>>> 2011-03-01 01:31:09+0800)) > > >>>> > > >>>> It's not a problem for the release, but I think maybe we can > > >>>> reduce annoying zk logs for this test to get around this problem. > > >>>> > > >>>> > > >>>> Best, > > >>>> Jiannan > > >>>> > > >>>> > > >>>> > > >>>> > > >>>> On 9/11/13 6:52 AM, "Ivan Kelly" <iv...@apache.org> wrote: > > >>>> > > >>>>> This is the first release candidate for Apache Bookkeeper, > > >>>>> version 4.2.2. > > >>>>> > > >>>>> This is a bugfix release for 4.2.1. There are some minor API > > >>>>> improvements. Notably, it is now possible to check whether a > > >>>>> ledger > > >>> is > > >>>>> closed without opening it, and it is now possible to get a list > > >>>>> of ledgers available in the cluster. > > >>>>> > > >>>>> The full release notes is available at: > > >>>>> > > >>>> > > >>>> https://issues.apache.org/jira/secure/ReleaseNote.jspa?version=12 > > >>>> 324601& > > >>>> st > > >>>>> yleName=Text&projectId=12311293 > > >>>>> > > >>>>> *** Please download, test and vote by September 16th 2013, 10:00 > > >>> UTC+0. > > >>>>> *** > > >>>>> > > >>>>> Note that we are voting upon the source (tag), binaries are > > >>>>> provided > > >>> for > > >>>>> convenience. > > >>>>> > > >>>>> Source and binary files: > > >>>>> http://people.apache.org/~ivank/bookkeeper-4.2.2-candidate-0/ > > >>>>> > > >>>>> Maven staging repo: > > >>>> > > >>>> https://repository.apache.org/content/repositories/orgapachebookk > > >>>> eeper-0 > > >>>> 25 > > >>>>> / > > >>>>> > > >>>>> The tag to be voted upon: > > >>>> > > >>>> > https://svn.apache.org/repos/asf/zookeeper/bookkeeper/tags/releas > > >>>> e-4.2.2 > > >>>>> > > >>>>> Bookkeeper's KEYS file containing PGP keys we use to sign the > > >>> release: > > >>>>> http://svn.apache.org/repos/asf/zookeeper/bookkeeper/dist/KEYS > > >>>>> > > >>>>> Please download the the source package, and follow the README to > > >>> build > > >>>>> and run a bookkeeper and hedwig service. > > >>>>> > > >>>>> -Ivan > > >>>> > > > > >