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> >>
