Taylor,
Lets do another release build since we have other jiras merged
in .
Thanks,
Harsha
On Mon, Apr 4, 2016, at 02:07 PM, P. Taylor Goetz wrote:
> FYI I’ve merged fixes for following to the 1.x-branch:
> * STORM-1670: LocalState#get(String) can throw FileNotFoundException
> which may result supervisor.clj#sync-processes stop assigning new
> workers/assignments
> * STORM-1677: Test resource files are excluded from source distribution,
> which makes logviewer-test failing
>
> Still waiting to hear opinions on whether we should cancel this vote and
> cut a new RC.
>
> I’m leaning toward a new RC at this point.
>
> -Taylor
>
> > On Apr 4, 2016, at 11:50 AM, P. Taylor Goetz <[email protected]> wrote:
> >
> > The question at this point is whether we want to cancel the release vote
> > and cut a new RC, or move forward with this one.
> >
> > I don’t see the test issue as critical — users will still be able to build
> > the software with the `-DskipTests=true` flag. I also feel that once the
> > 1.0 release is out we will go into a phase where we release updates (e.g.
> > 1.0.x, 1.x) at a relatively rapid rate.
> >
> > I could go either way. What does everyone else think?
> >
> > -Taylor
> >
> >
> >> On Apr 2, 2016, at 10:51 PM, Harsha <[email protected]> wrote:
> >>
> >> For rc2 please include STORM-1670 as well. Source distribution is
> >> important as well.
> >> -Harsha
> >>
> >> On Sat, Apr 2, 2016, at 01:02 AM, Jungtaek Lim wrote:
> >>> Here's my test,
> >>>
> >>> - testing with source distribution : FAIL
> >>> - unzip : OK
> >>> - building from source dist : FAIL
> >>> - how to build: running `mvn -P all-tests clean install` on unzipped
> >>> source dist.
> >>> - Test resource files (.log) are excluded from source distribution,
> >>> which makes logviewer-test failing.
> >>> - filed issue : https://issues.apache.org/jira/browse/STORM-1677
> >>> and
> >>> submit a patch via pull request
> >>> - after applying PR, tests are all passed.
> >>>
> >>> - testing with binary distribution (one machine) : OK
> >>> - launch daemons : OK
> >>> - run RollingTopWords (local) : OK (with minor issue)
> >>> - process doesn't terminate until killing manually or pressing Ctrl +
> >>> C
> >>> - 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
> >>> - run WordCountTopology (remote) : OK
> >>> - multiple workers and multi-lang bolt
> >>>
> >>> Minor but just leaving note: I saw /log (logviewer) throwing HTTP STATUS
> >>> 500 sometimes, but I can't reproduce it. At that time, deep-search
> >>> doesn't
> >>> work at that topology, too.
> >>>
> >>> Total
> >>> - source distribution: -1
> >>> - binary distribution: +1
> >>>
> >>> I don't know how it is important to build with source distribution, but
> >>> we
> >>> can address STORM-1677 and initiate voting RC2 immediately although we
> >>> think it's a 'blocker' issue.
> >>>
> >>> Thanks,
> >>> Jungtaek Lim (HeartSaVioR)
> >>>
> >>> 2016년 4월 2일 (토) 오전 8:50, P. Taylor Goetz <[email protected]>님이 작성:
> >>>
> >>>> This is a call to vote on releasing Apache Storm 1.0.0 (rc1)
> >>>>
> >>>> 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=787e4a6c375d290f724e59b3d8ebe34806ccd0d5
> >>>>
> >>>> 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=45b1b148401fd05f0f79cc7abdf6b5c7fc43df20;hb=d02f94268dec229d1125a24fdf53fa303cbc2b29
> >>>>
> >>>> The source archive being voted upon can be found here:
> >>>>
> >>>>
> >>>> https://dist.apache.org/repos/dist/dev/storm/apache-storm-1.0.0/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/
> >>>>
> >>>> 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-1027/
> >>>>
> >>>> 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. This is a major
> >>>> milestone of which we should all be proud.
> >>>>
> >>>> This is not an April fools joke. ;)
> >>>>
> >>>> -Taylor
> >>>>
> >
>
> Email had 1 attachment:
> + signature.asc
> 1k (application/pgp-signature)
