-1 (Non binding) on account of https://issues.apache.org/jira/browse/STORM-1693 'mvn clean install' - OK Topology submission/kill/activate/deactivate - OK thread dump - OK log search - OK (by the way, really helpful feature)
If others have run their topologies for a while, and are not hitting STORM-1693, it can be considered as non-blocker. On Fri, Apr 8, 2016 at 11:04 AM, Harsha <[email protected]> wrote: > +1 (binding) > Tested in a 3 node cluster with example topologies. > Checked the binaries. > > Thanks, > Harsha > > On Thu, Apr 7, 2016, at 10:02 PM, John Fang wrote: > > +1 > > > > Regards > > John Fang > > > > > > > > -----邮件原件----- > > 发件人: Bobby Evans [mailto:[email protected]] > > 发送时间: 2016年4月8日 2:36 > > 收件人: [email protected] > > 主题: Re: [VOTE] Release Apache Storm 1.0.0 (rc2) > > > > Forgot to say +1 binding > > - Bobby > > > > On Thursday, April 7, 2016 1:36 PM, Bobby Evans <[email protected] > > > > wrote: > > > > > > With the default configs everything seems to work fine. I have run into > > a few issues with not being able to set storm.workers.artifacts.dir, but > > I don't think it is a blocker. > > - Bobby > > > > On Wednesday, April 6, 2016 10:40 AM, Jungtaek Lim > > <[email protected]> wrote: > > > > > > +1 (binding) > > > > - testing with source distribution : OK > > - unzip : OK > > - building from source dist : OK > > - how to build: running `mvn -P all-tests clean install` on unzipped > > source dist. > > > > - testing with binary distribution (one machine) : OK > > - launch daemons : OK > > - run RollingTopWords (local) : OK > > - run RollingTopWords (remote) : OK > > - activate / deactivate / rebalance / kill : OK > > - logviewer (worker dir, daemon dir) : OK > > - change log level : OK > > - thread dump, heap dump, restart worker : OK > > - log search : OK > > - run WordCountTopology (remote) : OK > > - multiple workers and multi-lang bolt > > > > Amazing works everyone. > > Please note that we should check the site and fix the docs before > > announcing release. I saw some broken links today so I'll open another > > thread to check and fix them. > > > > Thanks, > > Jungtaek Lim (HeartSaVioR) > > > > ps. Found minor issues which I think they shouldn’t block releasing. Will > > raise issues when reproducible. > > > > - Process is not shutting down clearly in local mode, but ‘Ctrl + C’ > > can > > terminate the process. > > - when searching keyword in /logviewer_search.html with daemon log > > file, > > is-daemon=yes parameter is gone so search shows no result. > > - it works with /daemonlog, but since result page is > > /logviewer_search.html, user’s next search will failed. > > - When testing, build always fails on co-working space, cause logged > > host ip in console log was not the ip my dev machine was assigned. I > > expected private IP but logged ip seems to be public ip. Worker even > > try to > > connect to <public ip>:1024 and <public ip>:1025 which ports are also > > not > > expected. > > > > > > > > 2016년 4월 6일 (수) 오후 7:56, Satish Duggana <[email protected]>님이 작성: > > > > > > > > +1 (Non binding) > > > > > > - Retrieved source archive and built using 'mvn clean install -P > all-tests’ > > > - Checked Release notes etc. > > > - Built the binary package from the above source archive > > > - Ran different topologies in local cluster > > > - Created a 3 node cluster cluster with worker slots. > > > - Deployed few topologies > > > - Checked various options(like deactivate/kill/activate topology > view > > > etc) and monitoring stats in the UI for those topologies. > > > - Ran storm commands on those topologies like > > > deactivate/rebalance/activate/kill with respective options. > > > - Killed some of the workers to check failover etc. > > > - Checked the logs. > > > > > > > > > > > > > > > Thanks, > > > Satish. > > > > > > On 4/6/16, 3:49 PM, "Arun Iyer on behalf of Arun Mahadevan" < > > > [email protected] on behalf of [email protected]> wrote: > > > > > > >+1 > > > > > > > >Verified build from the source archive, deployed .tar.gz binary and > ran a > > > sample topology. > > > > > > > >- Arun > > > > > > > > > > > > > > > >On 4/6/16, 2:40 AM, "P. Taylor Goetz" <[email protected]> wrote: > > > > > > > >>+1 (binding) > > > >> > > > >>- Verified build from source archive with `mvn clean install -P > > > all-tests` > > > >>- Checked LICENSE and NOTICE files > > > >>- Deployed to a small cluster and tested a variety of topologies. > > > >> > > > >>-Taylor > > > >> > > > >>> On Apr 5, 2016, at 2:38 PM, P. Taylor Goetz <[email protected]> > > > wrote: > > > >>> > > > >>> This is a call to vote on releasing Apache Storm 1.0.0 (rc2) > > > >>> > > > >>> Full list of changes in this release: > > > >>> > > > >>> > > > > https://git-wip-us.apache.org/repos/asf?p=storm.git;a=blob_plain;f=CHANGELOG.md;hb=dba655a47aaad74f26b9bb9a75fa52c0eedd8b1e > > > >>> > > > >>> The tag/commit to be voted upon is v1.0.0: > > > >>> > > > >>> > > > > https://git-wip-us.apache.org/repos/asf?p=storm.git;a=tree;h=91db02dbcbfc4ce6b949b1316c9ebc9ec1bcc95f;hb=dba655a47aaad74f26b9bb9a75fa52c0eedd8b1e > > > >>> > > > >>> The source archive being voted upon can be found here: > > > >>> > > > >>> > > > > https://dist.apache.org/repos/dist/dev/storm/apache-storm-1.0.0-rc2/apache-storm-1.0.0-src.tar.gz > > > >>> > > > >>> Other release files, signatures and digests can be found here: > > > >>> > > > >>> > https://dist.apache.org/repos/dist/dev/storm/apache-storm-1.0.0-rc2/ > > > >>> > > > >>> The release artifacts are signed with the following key: > > > >>> > > > >>> > > > > https://git-wip-us.apache.org/repos/asf?p=storm.git;a=blob_plain;f=KEYS;hb=22b832708295fa2c15c4f3c70ac0d2bc6fded4bd > > > >>> > > > >>> The Nexus staging repository for this release is: > > > >>> > > > >>> > > > > https://repository.apache.org/content/repositories/orgapachestorm-1028/ > > > >>> > > > >>> Please vote on releasing this package as Apache Storm 1.0.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 1.0.0 > > > >>> [ ] 0 No opinion > > > >>> [ ] -1 Do not release this package because... > > > >>> > > > >>> Thanks to everyone who contributed to this release. > > > >>> > > > >>> -Taylor > > > >> > > > > > > > > > > > > > > > > > > > > > > -- Regards, Abhishek Agarwal
