Thanks, Mukul for making the release.

https://issues.apache.org/jira/browse/RATIS-642 was reopened.  We
probably should fix the NOTICE file first.

Tsz-Wo


On Thu, Aug 1, 2019 at 12:31 PM Ankit Singhal <[email protected]> wrote:
>
> +1
>
> -- Built from the source - OK
> -- Hashes and Signatures[5]  - OK
> -- Ran unit tests - All passed
> -- Ran rat check on the source - OK
>
> Some comments:-
> * NOTICE[1] assembled in a binary tar doesn't include a NOTICE[2] from the
> project.
> * NOTICE[1] doesn't look exhaustive (eg:- No mention of Junit dependency
> "Eclipse Public License 1.0") under Could also use a supplemental
> model[4] for dependencies which have incomplete information about their
> NOTICE/LICENSE
> * same with LICENSE[3], for eg:- no mention of "Eclipse Public License - v
> 1.0"[4]
>
> [1]
> http://people.apache.org/~msingh/0.4.0-rc1/apache-ratis-incubating-0.4.0-rc1-bin.tar.gz
> [2] https://github.com/apache/incubator-ratis/blob/master/NOTICE
> [3]
> http://people.apache.org/~msingh/0.4.0-rc1/apache-ratis-incubating-0.4.0-rc1-bin.tar.gz
> [4]
> https://maven.apache.org/plugins/maven-remote-resources-plugin/supplemental-models.html
> [5] https://dist.apache.org/repos/dist/release/incubator/ratis/KEYS
>
> Regards,
> Ankit Singhal
>
> On Thu, Aug 1, 2019 at 11:02 AM Uma gangumalla <[email protected]> wrote:
>
> > +1 (binding)
> >
> > built from source
> > LICENCE and NOTICE files looks good.
> > Sig and hashes are fine
> >
> > Few test timeouts[1] came in my 'mvn clean install'. Probably that's only
> > to me as others might already ran tests and +1ed. ( Should not be a blocker
> > I guess, if they are passing with others env)
> >
> > Regards,
> > Uma
> >
> > [1]
> > [INFO] Results:
> > [INFO]
> > [ERROR] Failures:
> > [ERROR]
> >
> > TestRaftSnapshotWithGrpc>RaftSnapshotBaseTest.testBasicInstallSnapshot:220->RaftSnapshotBaseTest.assertLeaderContent:78
> > [ERROR]
> >
> > TestRaftReconfigurationWithNetty>RaftReconfigurationBaseTest.testReconfTwice:140->RaftReconfigurationBaseTest.runTestReconfTwice:192->RaftReconfigurationBaseTest.lambda$runTestReconfTwice$5:200
> > s3: ConfigurationManager, init=-1: [s0:0.0.0.0:50941, s1:0.0.0.0:45664,
> > s2:
> > 0.0.0.0:49625], old=null, confs={
> >   0 -> 0: [s0:0.0.0.0:50941, s1:0.0.0.0:45664, s2:0.0.0.0:49625], old=null
> >   41 -> 41: [s3:0.0.0.0:45637, s4:0.0.0.0:50873, s0:0.0.0.0:50941, s1:
> > 0.0.0.0:45664, s2:0.0.0.0:49625], old=[s0:0.0.0.0:50941, s1:0.0.0.0:45664,
> > s2:0.0.0.0:49625]
> >   43 -> 43: [s3:0.0.0.0:45637, s4:0.0.0.0:50873, s0:0.0.0.0:50941, s1:
> > 0.0.0.0:45664, s2:0.0.0.0:49625], old=null
> >   45 -> 45: [s3:0.0.0.0:45637, s4:0.0.0.0:50873, s2:0.0.0.0:49625],
> > old=[s3:
> > 0.0.0.0:45637, s4:0.0.0.0:50873, s0:0.0.0.0:50941, s1:0.0.0.0:45664, s2:
> > 0.0.0.0:49625]
> >   47 -> 47: [s3:0.0.0.0:45637, s4:0.0.0.0:50873, s2:0.0.0.0:49625],
> > old=null
> >   49 -> 49: [s3:0.0.0.0:45637, s4:0.0.0.0:50873, s2:0.0.0.0:49625],
> > old=null
> >   50 -> 50: [s3:0.0.0.0:45637, s4:0.0.0.0:50873, s2:0.0.0.0:49625],
> > old=null
> > } expected:<7> but was:<8>
> > [ERROR]
> >
> > TestRaftSnapshotWithNetty>RaftSnapshotBaseTest.testRestartPeer:146->RaftSnapshotBaseTest.assertLeaderContent:78
> > [ERROR]
> >
> > TestRaftLogMetrics.testFlushMetric:78->runTestFlushMetric:93->assertFlushCount:110
> > expected:<5> but was:<4>
> > [ERROR]
> >
> > TestRaftSnapshotWithSimulatedRpc>RaftSnapshotBaseTest.testBasicInstallSnapshot:220->RaftSnapshotBaseTest.assertLeaderContent:78
> > [ERROR] Errors:
> > [ERROR]
> >
> > TestRaftReconfigurationWithGrpc>RaftReconfigurationBaseTest.testReconfTimeout:207->RaftReconfigurationBaseTest.runTestReconfTimeout:217
> > » NullPointer
> > [ERROR]
> >
> > TestRequestLimitAsyncWithGrpc>RequestLimitAsyncBaseTest.testWriteElementLimit:65->RequestLimitAsyncBaseTest.runTestWriteElementLimit:119->BaseTest.getWithDefaultTimeout:168
> > » Timeout
> > [ERROR]
> >
> > TestRaftStateMachineExceptionWithNetty>RaftStateMachineExceptionTests.testRetryOnExceptionDuringReplication:177
> > » NullPointer
> > [ERROR]
> >
> > TestRaftStateMachineExceptionWithSimulatedRpc>RaftStateMachineExceptionTests.testRetryOnExceptionDuringReplication:177
> > » NullPointer
> > [INFO]
> > [ERROR] Tests run: 241, Failures: 5, Errors: 4, Skipped: 4
> >
> > On Mon, Jul 29, 2019 at 2:59 AM Mukul Kumar Singh <
> > [email protected]>
> > wrote:
> >
> > > Hi Apache Ratis PPMC,
> > >
> > > I am calling a vote for Apache Ratis incubating Release 0.4.0 rc1.
> > > This vote is after fixing issues RATIS-642 & RATIS-621 in the rc0 vote.
> > >
> > > This is with respect to the discuss mailthread  <
> > >
> > https://lists.apache.org/thread.html/781fb6db8fe3327683f96b880496cf4b297f8666a2fab85c413c6f4b@%3Cdev.ratis.apache.org%3E
> > > >.
> > >
> > > The git tag to be voted upon:
> > >
> > >
> > https://gitbox.apache.org/repos/asf?p=incubator-ratis.git;a=shortlog;h=refs/tags/0.4.0-rc1
> > >
> > > The git commit hash:
> > >
> > >
> > https://gitbox.apache.org/repos/asf?p=incubator-ratis.git;a=commit;h=5dc4e717c8540b275cc2af33ab100c2a6ec21fc2
> > > <
> > >
> > https://gitbox.apache.org/repos/asf?p=incubator-ratis.git;a=commit;h=d4f5c9ded9ea4dc1a8acbec9a5e24f5e2f3bc6b6
> > > >
> > >
> > > The source and binary tarballs can be found at:
> > > http://people.apache.org/~msingh/0.4.0-rc1/  <
> > > http://people.apache.org/~msingh/0.4.0-rc0/>
> > >
> > > The fingerprint of key to sign release artifacts:
> > > 45B0 13F6 29C6 89CC 6EE3 77B0 8D43 C3D9 A8F1 02BD
> > >
> > > Release artifacts are signed with one of the keys available at:
> > > https://dist.apache.org/repos/dist/dev/incubator/ratis/KEYS  <
> > > https://dist.apache.org/repos/dist/dev/incubator/ratis/KEYS>
> > >
> > > This vote will remain open for at least 72 hours.
> > > Please vote on releasing this RC.  Thank you in advance.
> > >
> > > [ ] +1 approve
> > > [ ] +0 no opinion
> > > [ ] -1 disapprove (and reason why)
> > >
> > > Starting with my +1.
> > >
> > > Regards,
> > > Mukul Kumar Singh
> > >
> > >
> >

Reply via email to