Thanks Govind.

-Kishor

On Mon, Jun 15, 2020 at 2:03 PM Govind Menon <[email protected]>
wrote:

> Will close the vote and start a new one.
>
> On Mon, 15 Jun 2020 at 12:57, Ethan Li <[email protected]> wrote:
>
>> I cherry-picked the change to 2.2.x-branch. This unit test should be more
>> stable in environments with low cpu resources now. Thanks.
>>
>> > On Jun 15, 2020, at 12:34 PM, Kishorkumar Patil <
>> [email protected]> wrote:
>> >
>> > The rest of the tests are working fine. Manual running of throughput vs
>> > latency and other manual checks for DRPC work fine.
>> > The unit test *ExecutorTransferMultiThreadingTest* we added fails
>> > consistently for me on VM/mac for low CPU. So -1 on this release
>> candidate.
>> > Ethan has a patch to fix this -
>> > https://github.com/apache/storm/pull/3286/files. We  should cut the
>> next
>> > release candidate with this patch.
>> >
>> > [ERROR] Tests run: 1, Failures: 1, Errors: 0, Skipped: 0, Time
>> > elapsed: 1.582 s <<< FAILURE! - in
>> > org.apache.storm.executor.ExecutorTransferMultiThreadingTest
>> > [ERROR] testExecutorTransfer  Time elapsed: 1.582 s  <<< FAILURE!
>> > java.lang.AssertionError: expected:<10> but was:<0>
>> >       at org.junit.Assert.fail(Assert.java:88)
>> >       at org.junit.Assert.failNotEquals(Assert.java:834)
>> >       at org.junit.Assert.assertEquals(Assert.java:645)
>> >       at org.junit.Assert.assertEquals(Assert.java:631)
>> >       at
>> org.apache.storm.executor.ExecutorTransferMultiThreadingTest$SingleThreadedConsumer.finalCheck(ExecutorTransferMultiThreadingTest.java:187)
>> >       at
>> org.apache.storm.executor.ExecutorTransferMultiThreadingTest.testExecutorTransfer(ExecutorTransferMultiThreadingTest.java:136)
>> >       at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
>> >
>> > -Kishor
>> >
>> >
>> > On Mon, Jun 15, 2020 at 1:55 AM Julien Nioche <
>> [email protected]>
>> > wrote:
>> >
>> >> +1 (non binding)
>> >>
>> >> I have been running a crawl with StormCrawler successfully for the
>> last 2
>> >> days, no errors or exceptions to report. Thanks everyone!
>> >>
>> >> On Fri, 12 Jun 2020 at 00:03, Govind Menon <[email protected]> wrote:
>> >>
>> >>> This is a call to vote on releasing Apache Storm 2.2.0 (As a personal
>> >> note
>> >>> I would like to thank P. Taylor Goetz and Ethan Li for all the
>> >>> documentation they have set up to make releasing easier)
>> >>>
>> >>> Full list of changes in this release:
>> >>>
>> >>>
>> >>>
>> >>
>> https://dist.apache.org/repos/dist/dev/storm/apache-storm-2.2.0-rc1/RELEASE_NOTES.html
>> >>>
>> >>> The tag/commit to be voted upon is v2.2.0:
>> >>>
>> >>>
>> >>>
>> >>
>> https://gitbox.apache.org/repos/asf?p=storm.git;a=commit;h=de1f107e58bfe61aa43ae87e26f657f76e004a19
>> >>>
>> >>> The source archive being voted upon can be found here:
>> >>>
>> >>>
>> >>>
>> >>
>> https://dist.apache.org/repos/dist/dev/storm/apache-storm-2.2.0-rc1/apache-storm-2.2.0-src.tar.gz
>> >>>
>> >>> Other release files, signatures and digests can be found here:
>> >>>
>> >>> https://dist.apache.org/repos/dist/dev/storm/apache-storm-2.2.0-rc1/
>> >>>
>> >>> The release artifacts are signed with the following key:
>> >>>
>> >>> https://www.apache.org/dist/storm/KEYS
>> >>>
>> >>> The Nexus staging repository for this release is:
>> >>>
>> >>>
>> https://repository.apache.org/content/repositories/orgapachestorm-1094
>> >>>
>> >>> Please vote on releasing this package as Apache Storm 2.2.0.
>> >>>
>> >>> When voting, please list the actions taken to verify the release.
>> >>>
>> >>> This vote will be open for at least 72 hours.
>> >>>
>> >>> [ ] +1 Release this package as Apache Storm 2.2.0
>> >>> [ ]  0 No opinion
>> >>> [ ] -1 Do not release this package because...
>> >>>
>> >>> Thanks to everyone who contributed to this release.
>> >>>
>> >>
>> >>
>> >> --
>> >>
>> >> *Open Source Solutions for Text Engineering*
>> >>
>> >> http://www.digitalpebble.com
>> >> http://digitalpebble.blogspot.com/
>> >> #digitalpebble <http://twitter.com/digitalpebble>
>> >>
>>
>>

Reply via email to