Re: [VOTE] Release Apache Storm 1.2.0 (rc4)

2018-02-15 Thread P. Taylor Goetz
Apologies for the delay in release announcements. I’m working with infra on an 
issue that’s preventing updates to the Storm’s, website.

It looks like the gitpubsub process hung. So while the changes to the website 
are in place, they haven’t been published.

I’m inclined to wait until tomorrow to ask INFRA to what I call “poking 
gitpubsub”, rather than rush it and have a release announcement in the middle 
of the night.

-Taylor

> On Feb 7, 2018, at 5:24 PM, P. Taylor Goetz  wrote:
> 
> This is a call to vote on releasing Apache Storm 1.2.0 (rc4)
> 
> Note that the only difference between rc4 and rc3 is the fix for 
> https://issues.apache.org/jira/browse/STORM-2942 
> .
> 
> Full list of changes in this release:
> 
> https://dist.apache.org/repos/dist/dev/storm/apache-storm-1.2.0-rc4/RELEASE_NOTES.html
>  
> 
> 
> The tag/commit to be voted upon is v1.2.0:
> 
> https://git-wip-us.apache.org/repos/asf?p=storm.git;a=tree;h=cef4d49e222e53656f38c40d754d4f41799cd9a9;hb=2a0097f9a20b9df494caadb87c87d4e4db01a7ed
>  
> 
> 
> The source archive being voted upon can be found here:
> 
> https://dist.apache.org/repos/dist/dev/storm/apache-storm-1.2.0-rc4/apache-storm-1.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-1.2.0-rc4/ 
> 
> 
> 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-1058 
> 
> 
> Please vote on releasing this package as Apache Storm 1.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 1.2.0
> [ ]  0 No opinion
> [ ] -1 Do not release this package because...
> 
> Thanks to everyone who contributed to this release.
> 
> -Taylor



signature.asc
Description: Message signed with OpenPGP


[RESULT] [VOTE] Release Apache Storm 1.2.0 (rc4)

2018-02-12 Thread P. Taylor Goetz
This vote is now closed and passes with 4 binding +1 votes, 1 non-binding +1, 
and no 0 or -1 votes.

Vote tally (* indicates a binding vote):

+1:
- Jungtaek Lim*
- Satish Duggana*
- Arun Iyer*
- Alexandre Vermeerbergen
- Bobby Evans*

I will release the staged artifacts and announce the release after the 24 hour 
waiting period.

-Taylor

> On Feb 7, 2018, at 5:24 PM, P. Taylor Goetz  wrote:
> 
> This is a call to vote on releasing Apache Storm 1.2.0 (rc4)
> 
> Note that the only difference between rc4 and rc3 is the fix for 
> https://issues.apache.org/jira/browse/STORM-2942 
> .
> 
> Full list of changes in this release:
> 
> https://dist.apache.org/repos/dist/dev/storm/apache-storm-1.2.0-rc4/RELEASE_NOTES.html
>  
> 
> 
> The tag/commit to be voted upon is v1.2.0:
> 
> https://git-wip-us.apache.org/repos/asf?p=storm.git;a=tree;h=cef4d49e222e53656f38c40d754d4f41799cd9a9;hb=2a0097f9a20b9df494caadb87c87d4e4db01a7ed
>  
> 
> 
> The source archive being voted upon can be found here:
> 
> https://dist.apache.org/repos/dist/dev/storm/apache-storm-1.2.0-rc4/apache-storm-1.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-1.2.0-rc4/ 
> 
> 
> 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-1058 
> 
> 
> Please vote on releasing this package as Apache Storm 1.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 1.2.0
> [ ]  0 No opinion
> [ ] -1 Do not release this package because...
> 
> Thanks to everyone who contributed to this release.
> 
> -Taylor



signature.asc
Description: Message signed with OpenPGP


Re: [VOTE] Release Apache Storm 1.2.0 (rc4)

2018-02-12 Thread Bobby Evans
+1

I built from source all of the unit tests passed and some basic integration
tests passed too.

On Sat, Feb 10, 2018 at 12:00 PM Alexandre Vermeerbergen <
avermeerber...@gmail.com> wrote:

> Hello,
>
> +1 (non binding)
>
> Details: I almost got nuts when applying storm-kafka-client-1.2.0 RC4
> to our preproduction Storm cluster whose performances dropped quite a
> lot (at least x2 less throughput) ... but then I realized that I made
> the classic mistake of mixing two changes  : I had also upgraded our
> Kafka dependencies from Kafka Client 0.10.20.0 to Kafka  Client
> 1.0.0.0.
>
> Then I rollbacked our Kafka dependencies to Kafka 0.10.2.0 but I kept
> Storm 1.2.0 RC4 : since then (~10 hours ago) it's working smoothly.
>
> In particular, performances of Storm Kafka Client 1.2.0 RC4 having
> autocommit=true setting are equivalent to the ones of Storm Kafka
> Client 1.2.0 with
> .setProcessingGuarantee(ProcessingGuarantee.NO_GUARANTEE).
>
> Note : our preproduction Storm cluster consumes & produces data in a
> Kafka Cluster based on Kafka 1.0.0. I guess there's some details we
> missed when using Kafka Client 1.0.0 libraries instead of Kafka Client
> 0.10.2.0, but I think this is something we can study outside this
> 1.2.0 voting process, because consuming & producing Kafka messages
> against a Kafka cluster based on Kafka 1.0.0 Brokers is considered as
> compatible by Kakfa documentation.
>
> Since we want to move to Kafka Client 1.0.0 at some time (in
> particular, to take advantage of official JRE 9 support of this Kafka
> version), we will keep on testing our system with this newer Kafka
> version and we will feeback to Storm dev list whatever the outcome is
> (I'm guessing some property changes from 0.10.2 to 1.0.0)
>
> Thank you very much to all Storm developers for bringing this great
> Storm 1.2.0 to us, and for your taking into account my feedbacks: I
> have no more bad news for 1.2.0 :)
>
>
> Best regards,
> Alexandre Vermeerbergen
>
> PS: just one question: is there a way to access 1.2.0 RC4 Storm
> documentation before it's switched on public Storm site?
> PPS: I saw Stig's feedback about our deprecated usage of
> auto.commit.internal.ms, but I haven't yet tried the "new way",
> meaning that it has no impact on our big scale test. But I will try it
> later.
>
> 2018-02-09 9:18 GMT+01:00 Alexandre Vermeerbergen <
> avermeerber...@gmail.com>:
> > Hello,
> >
> > For your information, yesterday I upgraded our preproduction cluster
> > from Storm 1.2.0 SNAPSHOT of December 2017 to 1.2.0 Release Candidate
> > 4.
> > Our alerting system based on Storm behaved quite bad with 1.2.0 RC4,
> > and this morning, one of our team member noticed this message in our
> > topologie's startup logs:
> >
> > 1279 [main] WARN  o.a.s.k.s.KafkaSpoutConfig - The KafkaConsumer
> > enable.auto.commit setting is not supported. You can configure similar
> > behavior through KafkaSpoutConfig.Builder.setProcessingGuarantee.This
> > will be treated as an error in the next major release.
> >
> > 1279 [main] INFO  o.a.s.k.s.KafkaSpoutConfig - Setting Kafka consumer
> > property 'enable.auto.commit' to 'false', because the spout does not
> > support auto-commit
> >
> >
> > So I will resume our tests with this new setting and I will feedback
> > once I have enought uptime on our "large preproduction cluster" with
> > 1.2.0 RC4.
> >
> > Note: may I suggest to make this breaking change visible in Storm
> > 1.2.0 releases notes? this is quite impacting. Or event better: make
> > the topologies unable to start when they use such a removed property,
> > so that at least people aren't fooled until they wonder why their
> > Kafka spouts aren't anymore behaving like before and check logs?
> >
> > More to come when my preproduction tests will have been completed (1or
> > 2 days needed).
> >
> > Best regards,
> > Alexandre
> >
> > 2018-02-07 23:24 GMT+01:00 P. Taylor Goetz :
> >> This is a call to vote on releasing Apache Storm 1.2.0 (rc4)
> >>
> >> Note that the only difference between rc4 and rc3 is the fix for
> >> https://issues.apache.org/jira/browse/STORM-2942.
> >>
> >> Full list of changes in this release:
> >>
> >>
> https://dist.apache.org/repos/dist/dev/storm/apache-storm-1.2.0-rc4/RELEASE_NOTES.html
> >>
> >> The tag/commit to be voted upon is v1.2.0:
> >>
> >>
> https://git-wip-us.apache.org/repos/asf?p=storm.git;a=tree;h=cef4d49e222e53656f38c40d754d4f41799cd9a9;hb=2a0097f9a20b9df494caadb87c87d4e4db01a7ed
> >>
> >> The source archive being voted upon can be found here:
> >>
> >>
> https://dist.apache.org/repos/dist/dev/storm/apache-storm-1.2.0-rc4/apache-storm-1.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-1.2.0-rc4/
> >>
> >> The release artifacts are signed with the following key:
> >>
> >>
> 

Re: [VOTE] Release Apache Storm 1.2.0 (rc4)

2018-02-10 Thread Alexandre Vermeerbergen
Hello,

+1 (non binding)

Details: I almost got nuts when applying storm-kafka-client-1.2.0 RC4
to our preproduction Storm cluster whose performances dropped quite a
lot (at least x2 less throughput) ... but then I realized that I made
the classic mistake of mixing two changes  : I had also upgraded our
Kafka dependencies from Kafka Client 0.10.20.0 to Kafka  Client
1.0.0.0.

Then I rollbacked our Kafka dependencies to Kafka 0.10.2.0 but I kept
Storm 1.2.0 RC4 : since then (~10 hours ago) it's working smoothly.

In particular, performances of Storm Kafka Client 1.2.0 RC4 having
autocommit=true setting are equivalent to the ones of Storm Kafka
Client 1.2.0 with
.setProcessingGuarantee(ProcessingGuarantee.NO_GUARANTEE).

Note : our preproduction Storm cluster consumes & produces data in a
Kafka Cluster based on Kafka 1.0.0. I guess there's some details we
missed when using Kafka Client 1.0.0 libraries instead of Kafka Client
0.10.2.0, but I think this is something we can study outside this
1.2.0 voting process, because consuming & producing Kafka messages
against a Kafka cluster based on Kafka 1.0.0 Brokers is considered as
compatible by Kakfa documentation.

Since we want to move to Kafka Client 1.0.0 at some time (in
particular, to take advantage of official JRE 9 support of this Kafka
version), we will keep on testing our system with this newer Kafka
version and we will feeback to Storm dev list whatever the outcome is
(I'm guessing some property changes from 0.10.2 to 1.0.0)

Thank you very much to all Storm developers for bringing this great
Storm 1.2.0 to us, and for your taking into account my feedbacks: I
have no more bad news for 1.2.0 :)


Best regards,
Alexandre Vermeerbergen

PS: just one question: is there a way to access 1.2.0 RC4 Storm
documentation before it's switched on public Storm site?
PPS: I saw Stig's feedback about our deprecated usage of
auto.commit.internal.ms, but I haven't yet tried the "new way",
meaning that it has no impact on our big scale test. But I will try it
later.

2018-02-09 9:18 GMT+01:00 Alexandre Vermeerbergen :
> Hello,
>
> For your information, yesterday I upgraded our preproduction cluster
> from Storm 1.2.0 SNAPSHOT of December 2017 to 1.2.0 Release Candidate
> 4.
> Our alerting system based on Storm behaved quite bad with 1.2.0 RC4,
> and this morning, one of our team member noticed this message in our
> topologie's startup logs:
>
> 1279 [main] WARN  o.a.s.k.s.KafkaSpoutConfig - The KafkaConsumer
> enable.auto.commit setting is not supported. You can configure similar
> behavior through KafkaSpoutConfig.Builder.setProcessingGuarantee.This
> will be treated as an error in the next major release.
>
> 1279 [main] INFO  o.a.s.k.s.KafkaSpoutConfig - Setting Kafka consumer
> property 'enable.auto.commit' to 'false', because the spout does not
> support auto-commit
>
>
> So I will resume our tests with this new setting and I will feedback
> once I have enought uptime on our "large preproduction cluster" with
> 1.2.0 RC4.
>
> Note: may I suggest to make this breaking change visible in Storm
> 1.2.0 releases notes? this is quite impacting. Or event better: make
> the topologies unable to start when they use such a removed property,
> so that at least people aren't fooled until they wonder why their
> Kafka spouts aren't anymore behaving like before and check logs?
>
> More to come when my preproduction tests will have been completed (1or
> 2 days needed).
>
> Best regards,
> Alexandre
>
> 2018-02-07 23:24 GMT+01:00 P. Taylor Goetz :
>> This is a call to vote on releasing Apache Storm 1.2.0 (rc4)
>>
>> Note that the only difference between rc4 and rc3 is the fix for
>> https://issues.apache.org/jira/browse/STORM-2942.
>>
>> Full list of changes in this release:
>>
>> https://dist.apache.org/repos/dist/dev/storm/apache-storm-1.2.0-rc4/RELEASE_NOTES.html
>>
>> The tag/commit to be voted upon is v1.2.0:
>>
>> https://git-wip-us.apache.org/repos/asf?p=storm.git;a=tree;h=cef4d49e222e53656f38c40d754d4f41799cd9a9;hb=2a0097f9a20b9df494caadb87c87d4e4db01a7ed
>>
>> The source archive being voted upon can be found here:
>>
>> https://dist.apache.org/repos/dist/dev/storm/apache-storm-1.2.0-rc4/apache-storm-1.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-1.2.0-rc4/
>>
>> 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-1058
>>
>> Please vote on releasing this package as Apache Storm 1.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 1.2.0
>> [ ]  0 No 

Re: [VOTE] Release Apache Storm 1.2.0 (rc4)

2018-02-10 Thread Stig Rohde Døssing
Alexandre,

The warning you noted is just saying that enable.auto.commit is not
supported and you should use ProcessingGuarantee.NO_GUARANTEE instead. The
method logging that warning also sets the guarantee to NO_GUARANTEE, so I
don't expect you to see any difference between enable.auto.commit=true and
setting ProcessingGuarantee.NO_GUARANTEE. It shouldn't be a breaking change.

Can you elaborate what you mean when you say the topology behaves badly?

It also occurs to me that the Kafka setting auto.commit.interval.ms
(default: 5 seconds)  is now ignored, and you need to instead use
setOffsetCommitPeriodMs on the KafkaSpoutConfig.Builder (default: 30
seconds). Maybe that's the change causing you issues?

2018-02-10 1:18 GMT+01:00 Arun Iyer :

> +1 (binding)
>
> - Downloaded and built the source distribution
> - Downloaded the binary distributions
> - Validated MD5
> - Complied starter topologies from binary distribution
> - Ran ExclamationTopology and RollingTopWords
> - Checked the metrics from Storm UI
> - Check logs via log viewer
> - Listed and Killed topologies from UI and command line
>
> Thanks,
> Arun
>
>
> On 2/7/18, 8:16 PM, "Satish Duggana"  wrote:
>
> >+1 (binding)
> >
> >src distribution
> >  - Retrieved source archive and verified files
> >  - Built the binary package from the above source archive
> >
> >bin distribution
> >  - Ran different topologies in local cluster
> >  - Created a 3 node 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 change log level settings for topologies.
> >
> >Thanks,
> >Satish.
> >
> >On Thu, Feb 8, 2018 at 6:39 AM, Jungtaek Lim  wrote:
> >
> >> +1 (binding)
> >>
> >> > source
> >>
> >> - verify file (signature, MD5, SHA)
> >> -- source, tar.gz : OK
> >> -- source, zip : OK
> >>
> >> - extract file
> >> -- source, tar.gz : OK
> >> -- source, zip : OK
> >>
> >> - diff-ing extracted files between tar.gz and zip : OK
> >>
> >> - build source with JDK 7
> >> -- source, tar.gz : OK
> >>
> >> > binary
> >>
> >> - verify file (signature, MD5, SHA)
> >> -- binary, tar.gz : OK
> >> -- binary, zip : OK
> >>
> >> - extract file
> >> -- binary, tar.gz : OK
> >> -- binary, zip : OK
> >>
> >> - diff-ing extracted files between tar.gz and zip : OK
> >>
> >> I already verified others from RC3, and also verified that
> >> storm-kafka-monitor source/javadoc files are no longer exist in toollib
> >> directory.
> >>
> >> Thanks,
> >> Jungtaek Lim (HeartSaVioR)
> >>
> >> 2018년 2월 8일 (목) 오전 7:24, P. Taylor Goetz 님이 작성:
> >>
> >> > This is a call to vote on releasing Apache Storm 1.2.0 (rc4)
> >> >
> >> > Note that the only difference between rc4 and rc3 is the fix for
> >> > https://issues.apache.org/jira/browse/STORM-2942.
> >> >
> >> > Full list of changes in this release:
> >> >
> >> >
> >> > https://dist.apache.org/repos/dist/dev/storm/apache-storm-1.
> >> 2.0-rc4/RELEASE_NOTES.html
> >> >
> >> > The tag/commit to be voted upon is v1.2.0:
> >> >
> >> >
> >> > https://git-wip-us.apache.org/repos/asf?p=storm.git;a=tree;h=
> >> cef4d49e222e53656f38c40d754d4f41799cd9a9;hb=
> 2a0097f9a20b9df494caadb87c87d4
> >> e4db01a7ed
> >> >
> >> > The source archive being voted upon can be found here:
> >> >
> >> >
> >> > https://dist.apache.org/repos/dist/dev/storm/apache-storm-1.
> >> 2.0-rc4/apache-storm-1.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-1.2.0-rc4/
> >> >
> >> > 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-1058
> >> >
> >> > Please vote on releasing this package as Apache Storm 1.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 1.2.0
> >> > [ ]  0 No opinion
> >> > [ ] -1 Do not release this package because...
> >> >
> >> > Thanks to everyone who contributed to this release.
> >> >
> >> > -Taylor
> >> >
> >>
>


Re: [VOTE] Release Apache Storm 1.2.0 (rc4)

2018-02-09 Thread Arun Iyer
+1 (binding)

- Downloaded and built the source distribution
- Downloaded the binary distributions
- Validated MD5
- Complied starter topologies from binary distribution
- Ran ExclamationTopology and RollingTopWords
- Checked the metrics from Storm UI
- Check logs via log viewer
- Listed and Killed topologies from UI and command line

Thanks,
Arun


On 2/7/18, 8:16 PM, "Satish Duggana"  wrote:

>+1 (binding)
>
>src distribution
>  - Retrieved source archive and verified files
>  - Built the binary package from the above source archive
>
>bin distribution
>  - Ran different topologies in local cluster
>  - Created a 3 node 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 change log level settings for topologies.
>
>Thanks,
>Satish.
>
>On Thu, Feb 8, 2018 at 6:39 AM, Jungtaek Lim  wrote:
>
>> +1 (binding)
>>
>> > source
>>
>> - verify file (signature, MD5, SHA)
>> -- source, tar.gz : OK
>> -- source, zip : OK
>>
>> - extract file
>> -- source, tar.gz : OK
>> -- source, zip : OK
>>
>> - diff-ing extracted files between tar.gz and zip : OK
>>
>> - build source with JDK 7
>> -- source, tar.gz : OK
>>
>> > binary
>>
>> - verify file (signature, MD5, SHA)
>> -- binary, tar.gz : OK
>> -- binary, zip : OK
>>
>> - extract file
>> -- binary, tar.gz : OK
>> -- binary, zip : OK
>>
>> - diff-ing extracted files between tar.gz and zip : OK
>>
>> I already verified others from RC3, and also verified that
>> storm-kafka-monitor source/javadoc files are no longer exist in toollib
>> directory.
>>
>> Thanks,
>> Jungtaek Lim (HeartSaVioR)
>>
>> 2018년 2월 8일 (목) 오전 7:24, P. Taylor Goetz 님이 작성:
>>
>> > This is a call to vote on releasing Apache Storm 1.2.0 (rc4)
>> >
>> > Note that the only difference between rc4 and rc3 is the fix for
>> > https://issues.apache.org/jira/browse/STORM-2942.
>> >
>> > Full list of changes in this release:
>> >
>> >
>> > https://dist.apache.org/repos/dist/dev/storm/apache-storm-1.
>> 2.0-rc4/RELEASE_NOTES.html
>> >
>> > The tag/commit to be voted upon is v1.2.0:
>> >
>> >
>> > https://git-wip-us.apache.org/repos/asf?p=storm.git;a=tree;h=
>> cef4d49e222e53656f38c40d754d4f41799cd9a9;hb=2a0097f9a20b9df494caadb87c87d4
>> e4db01a7ed
>> >
>> > The source archive being voted upon can be found here:
>> >
>> >
>> > https://dist.apache.org/repos/dist/dev/storm/apache-storm-1.
>> 2.0-rc4/apache-storm-1.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-1.2.0-rc4/
>> >
>> > 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-1058
>> >
>> > Please vote on releasing this package as Apache Storm 1.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 1.2.0
>> > [ ]  0 No opinion
>> > [ ] -1 Do not release this package because...
>> >
>> > Thanks to everyone who contributed to this release.
>> >
>> > -Taylor
>> >
>>


Re: [VOTE] Release Apache Storm 1.2.0 (rc4)

2018-02-09 Thread Alexandre Vermeerbergen
Hello,

For your information, yesterday I upgraded our preproduction cluster
from Storm 1.2.0 SNAPSHOT of December 2017 to 1.2.0 Release Candidate
4.
Our alerting system based on Storm behaved quite bad with 1.2.0 RC4,
and this morning, one of our team member noticed this message in our
topologie's startup logs:

1279 [main] WARN  o.a.s.k.s.KafkaSpoutConfig - The KafkaConsumer
enable.auto.commit setting is not supported. You can configure similar
behavior through KafkaSpoutConfig.Builder.setProcessingGuarantee.This
will be treated as an error in the next major release.

1279 [main] INFO  o.a.s.k.s.KafkaSpoutConfig - Setting Kafka consumer
property 'enable.auto.commit' to 'false', because the spout does not
support auto-commit


So I will resume our tests with this new setting and I will feedback
once I have enought uptime on our "large preproduction cluster" with
1.2.0 RC4.

Note: may I suggest to make this breaking change visible in Storm
1.2.0 releases notes? this is quite impacting. Or event better: make
the topologies unable to start when they use such a removed property,
so that at least people aren't fooled until they wonder why their
Kafka spouts aren't anymore behaving like before and check logs?

More to come when my preproduction tests will have been completed (1or
2 days needed).

Best regards,
Alexandre

2018-02-07 23:24 GMT+01:00 P. Taylor Goetz :
> This is a call to vote on releasing Apache Storm 1.2.0 (rc4)
>
> Note that the only difference between rc4 and rc3 is the fix for
> https://issues.apache.org/jira/browse/STORM-2942.
>
> Full list of changes in this release:
>
> https://dist.apache.org/repos/dist/dev/storm/apache-storm-1.2.0-rc4/RELEASE_NOTES.html
>
> The tag/commit to be voted upon is v1.2.0:
>
> https://git-wip-us.apache.org/repos/asf?p=storm.git;a=tree;h=cef4d49e222e53656f38c40d754d4f41799cd9a9;hb=2a0097f9a20b9df494caadb87c87d4e4db01a7ed
>
> The source archive being voted upon can be found here:
>
> https://dist.apache.org/repos/dist/dev/storm/apache-storm-1.2.0-rc4/apache-storm-1.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-1.2.0-rc4/
>
> 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-1058
>
> Please vote on releasing this package as Apache Storm 1.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 1.2.0
> [ ]  0 No opinion
> [ ] -1 Do not release this package because...
>
> Thanks to everyone who contributed to this release.
>
> -Taylor


Re: [VOTE] Release Apache Storm 1.2.0 (rc4)

2018-02-07 Thread Satish Duggana
+1 (binding)

src distribution
  - Retrieved source archive and verified files
  - Built the binary package from the above source archive

bin distribution
  - Ran different topologies in local cluster
  - Created a 3 node 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 change log level settings for topologies.

Thanks,
Satish.

On Thu, Feb 8, 2018 at 6:39 AM, Jungtaek Lim  wrote:

> +1 (binding)
>
> > source
>
> - verify file (signature, MD5, SHA)
> -- source, tar.gz : OK
> -- source, zip : OK
>
> - extract file
> -- source, tar.gz : OK
> -- source, zip : OK
>
> - diff-ing extracted files between tar.gz and zip : OK
>
> - build source with JDK 7
> -- source, tar.gz : OK
>
> > binary
>
> - verify file (signature, MD5, SHA)
> -- binary, tar.gz : OK
> -- binary, zip : OK
>
> - extract file
> -- binary, tar.gz : OK
> -- binary, zip : OK
>
> - diff-ing extracted files between tar.gz and zip : OK
>
> I already verified others from RC3, and also verified that
> storm-kafka-monitor source/javadoc files are no longer exist in toollib
> directory.
>
> Thanks,
> Jungtaek Lim (HeartSaVioR)
>
> 2018년 2월 8일 (목) 오전 7:24, P. Taylor Goetz 님이 작성:
>
> > This is a call to vote on releasing Apache Storm 1.2.0 (rc4)
> >
> > Note that the only difference between rc4 and rc3 is the fix for
> > https://issues.apache.org/jira/browse/STORM-2942.
> >
> > Full list of changes in this release:
> >
> >
> > https://dist.apache.org/repos/dist/dev/storm/apache-storm-1.
> 2.0-rc4/RELEASE_NOTES.html
> >
> > The tag/commit to be voted upon is v1.2.0:
> >
> >
> > https://git-wip-us.apache.org/repos/asf?p=storm.git;a=tree;h=
> cef4d49e222e53656f38c40d754d4f41799cd9a9;hb=2a0097f9a20b9df494caadb87c87d4
> e4db01a7ed
> >
> > The source archive being voted upon can be found here:
> >
> >
> > https://dist.apache.org/repos/dist/dev/storm/apache-storm-1.
> 2.0-rc4/apache-storm-1.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-1.2.0-rc4/
> >
> > 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-1058
> >
> > Please vote on releasing this package as Apache Storm 1.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 1.2.0
> > [ ]  0 No opinion
> > [ ] -1 Do not release this package because...
> >
> > Thanks to everyone who contributed to this release.
> >
> > -Taylor
> >
>


Re: [VOTE] Release Apache Storm 1.2.0 (rc4)

2018-02-07 Thread Jungtaek Lim
+1 (binding)

> source

- verify file (signature, MD5, SHA)
-- source, tar.gz : OK
-- source, zip : OK

- extract file
-- source, tar.gz : OK
-- source, zip : OK

- diff-ing extracted files between tar.gz and zip : OK

- build source with JDK 7
-- source, tar.gz : OK

> binary

- verify file (signature, MD5, SHA)
-- binary, tar.gz : OK
-- binary, zip : OK

- extract file
-- binary, tar.gz : OK
-- binary, zip : OK

- diff-ing extracted files between tar.gz and zip : OK

I already verified others from RC3, and also verified that
storm-kafka-monitor source/javadoc files are no longer exist in toollib
directory.

Thanks,
Jungtaek Lim (HeartSaVioR)

2018년 2월 8일 (목) 오전 7:24, P. Taylor Goetz 님이 작성:

> This is a call to vote on releasing Apache Storm 1.2.0 (rc4)
>
> Note that the only difference between rc4 and rc3 is the fix for
> https://issues.apache.org/jira/browse/STORM-2942.
>
> Full list of changes in this release:
>
>
> https://dist.apache.org/repos/dist/dev/storm/apache-storm-1.2.0-rc4/RELEASE_NOTES.html
>
> The tag/commit to be voted upon is v1.2.0:
>
>
> https://git-wip-us.apache.org/repos/asf?p=storm.git;a=tree;h=cef4d49e222e53656f38c40d754d4f41799cd9a9;hb=2a0097f9a20b9df494caadb87c87d4e4db01a7ed
>
> The source archive being voted upon can be found here:
>
>
> https://dist.apache.org/repos/dist/dev/storm/apache-storm-1.2.0-rc4/apache-storm-1.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-1.2.0-rc4/
>
> 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-1058
>
> Please vote on releasing this package as Apache Storm 1.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 1.2.0
> [ ]  0 No opinion
> [ ] -1 Do not release this package because...
>
> Thanks to everyone who contributed to this release.
>
> -Taylor
>


[VOTE] Release Apache Storm 1.2.0 (rc4)

2018-02-07 Thread P. Taylor Goetz
This is a call to vote on releasing Apache Storm 1.2.0 (rc4)

Note that the only difference between rc4 and rc3 is the fix for 
https://issues.apache.org/jira/browse/STORM-2942 
.

Full list of changes in this release:

https://dist.apache.org/repos/dist/dev/storm/apache-storm-1.2.0-rc4/RELEASE_NOTES.html

The tag/commit to be voted upon is v1.2.0:

https://git-wip-us.apache.org/repos/asf?p=storm.git;a=tree;h=cef4d49e222e53656f38c40d754d4f41799cd9a9;hb=2a0097f9a20b9df494caadb87c87d4e4db01a7ed

The source archive being voted upon can be found here:

https://dist.apache.org/repos/dist/dev/storm/apache-storm-1.2.0-rc4/apache-storm-1.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-1.2.0-rc4/

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

Please vote on releasing this package as Apache Storm 1.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 1.2.0
[ ]  0 No opinion
[ ] -1 Do not release this package because...

Thanks to everyone who contributed to this release.

-Taylor


signature.asc
Description: Message signed with OpenPGP