I'm not 100% sure either, but I assume Storm serializes topology
information into thrift structure (at least Grouping.fields() takes
List<String> instead of Fields), so unless nodes are using different
versions of Storm (which we never support), that would be safe. If it's not
safe, we already broke rolling upgrade from under 1.1.0 to 1.1.0 since same
modification was done from 1.1.0.

2018년 2월 9일 (금) 오전 7:37, Erik Weathers <[email protected]>님이 작성:

> Something I just realized -- in the storm-kafka-client stomping into
> 1.0.x-branch PR, I backported a change to Fields.java which added a
> serialVersionUID.
> Could that potentially break topologies when you upgrade storm-core on the
> servers (nimbus, workers) from 1.0.{1..5} to 1.0.6?   I'm not super
> familiar with the serialization that occurs in Storm and whether that could
> break people.
>
>
> https://github.com/apache/storm/pull/2550/files#diff-71a428d508c4f5af0bfe3cc186e8edcf
>
> - Erik
>
> On Thu, Feb 8, 2018 at 1:25 PM, Bobby Evans <[email protected]>
> wrote:
>
> > +1 I built the code from the git tag, ran all the unit tests (which
> passed
> > the first time), and ran some tests on a single node cluster.
> >
> > It all looked good.
> >
> > - Bobby
> >
> > On Thu, Feb 8, 2018 at 1:22 PM P. Taylor Goetz <[email protected]>
> wrote:
> >
> > > This is a call to vote on releasing Apache Storm 1.0.6 (rc3)
> > >
> > > Full list of changes in this release:
> > >
> > >
> > > https://dist.apache.org/repos/dist/dev/storm/apache-storm-1.
> > 0.6-rc3/RELEASE_NOTES.html
> > >
> > > The tag/commit to be voted upon is v1.0.6:
> > >
> > >
> > > https://git-wip-us.apache.org/repos/asf?p=storm.git;a=tree;h=
> >
> e68365f9f947ddd1794b2edef2149fdfaa1590a2;hb=7993db01580ce62d44866dc00e0a72
> > 66984638d0
> > >
> > > The source archive being voted upon can be found here:
> > >
> > >
> > > https://dist.apache.org/repos/dist/dev/storm/apache-storm-1.
> > 0.6-rc3/apache-storm-1.0.6-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.6-rc3/
> > >
> > > 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-1060
> > >
> > > Please vote on releasing this package as Apache Storm 1.0.6.
> > >
> > > 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.6
> > > [ ]  0 No opinion
> > > [ ] -1 Do not release this package because...
> > >
> > > Thanks to everyone who contributed to this release.
> > >
> > > -Taylor
> > >
> >
>

Reply via email to