+1 (Non binding)

- testing with source distribution(built using `mvn -P all-tests clean
install`) : OK
- 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
    - log search : OK
    - thread dump, heap dump, restart worker : OK

I found links in `bin/storm.py` are unavailable,  but I don't think it is a
blocker.

BTW, I field a JIRA(STORM-1658) for improving and fixing documents(these
will be published to asf site).

Great work, Thanks everyone!


2016-04-08 13:02 GMT+08:00 John Fang <[email protected]>:

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

Reply via email to