Re: [VOTE] Release Apache Storm 2.0.0 RC7

2019-05-04 Thread P. Taylor Goetz
Yeah, I had an expiration date on now of my keys. I’ve sorted it out, but need 
to wait for propagation of myyl keys.

This is currently blocking the 1.2.x. and 1.1.x releases. But it looks like we 
can move forward with the 2.0.0 release.

-Taylor

> On May 2, 2019, at 4:51 PM, Ethan Li  wrote:
> 
> +1
> 
> 
> 1. Checked *.asc, *.md5, *.sha512
> 2. Built from the source code and ran unit tests; created a storm 
> distribution based on it
> 3. set up a standalone cluster (based on apache-storm-2.0.0.zip, 
> apache-storm-2.0.0.tar.gz, the storm distribution created above, separately)
> 4. Launched WordCountTopology and ThroughputVslatency and checked logs, 
> output, UI
> 4. Tested jstack, heap dump, deactivate, activate, rebalance, change log 
> level and kll on UI.
> 
> 
> I noticed "Note: This key has expired!” when I used "gpg —verify" to verify 
> the signature. I am not sure if we care about that.
> 
> 
> Thanks
> 
> - Ethan
> 
>> On Apr 29, 2019, at 5:49 PM, P. Taylor Goetz  wrote:
>> 
>> This is a call to vote on releasing Apache Storm 2.0.0 (rc7)
>> 
>> Full list of changes in this release:
>> 
>> https://dist.apache.org/repos/dist/dev/storm/apache-storm-2.0.0-rc7/RELEASE_NOTES.html
>> 
>> The tag/commit to be voted upon is v2.0.0:
>> 
>> https://git-wip-us.apache.org/repos/asf?p=storm.git;a=tree;h=007863edd95e838b3df414928c6fa3f28244ab49;hb=2ba95bbd1c911d4fc6363b1c4b9c4c6d86ac9aae
>> 
>> The source archive being voted upon can be found here:
>> 
>> https://dist.apache.org/repos/dist/dev/storm/apache-storm-2.0.0-rc7/apache-storm-2.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-2.0.0-rc7/
>> 
>> 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-1079
>> 
>> Please vote on releasing this package as Apache Storm 2.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 2.0.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 2.0.0 RC7

2019-05-02 Thread Ethan Li
+1


1. Checked *.asc, *.md5, *.sha512
2. Built from the source code and ran unit tests; created a storm distribution 
based on it
3. set up a standalone cluster (based on apache-storm-2.0.0.zip, 
apache-storm-2.0.0.tar.gz, the storm distribution created above, separately)
4. Launched WordCountTopology and ThroughputVslatency and checked logs, output, 
UI
4. Tested jstack, heap dump, deactivate, activate, rebalance, change log level 
and kll on UI.


I noticed "Note: This key has expired!” when I used "gpg —verify" to verify the 
signature. I am not sure if we care about that.


Thanks

- Ethan

> On Apr 29, 2019, at 5:49 PM, P. Taylor Goetz  wrote:
> 
> This is a call to vote on releasing Apache Storm 2.0.0 (rc7)
> 
> Full list of changes in this release:
> 
> https://dist.apache.org/repos/dist/dev/storm/apache-storm-2.0.0-rc7/RELEASE_NOTES.html
> 
> The tag/commit to be voted upon is v2.0.0:
> 
> https://git-wip-us.apache.org/repos/asf?p=storm.git;a=tree;h=007863edd95e838b3df414928c6fa3f28244ab49;hb=2ba95bbd1c911d4fc6363b1c4b9c4c6d86ac9aae
> 
> The source archive being voted upon can be found here:
> 
> https://dist.apache.org/repos/dist/dev/storm/apache-storm-2.0.0-rc7/apache-storm-2.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-2.0.0-rc7/
> 
> 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-1079
> 
> Please vote on releasing this package as Apache Storm 2.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 2.0.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 2.0.0 RC7

2019-05-01 Thread Stig Rohde Døssing
Subscription has been replaced with TopicFilter. Subscription isn't
deprecated in 1.x because it is not possible to backport TopicFilter
without breaking changes. The replacement is the TopicFilter and
ManualPartitioner classes.

getKey/ValueDeserializer is deprecated in the latest 1.x release, the
Javadoc will say how to replace it.

For convenience: " Please use {@link #getKafkaProps() } and look up the
entry for {@link ConsumerConfig#KEY_DESERIALIZER_CLASS_CONFIG} instead".
There's a similar one for the value deserializer.

I suspect the last error is because getValueDeserializer no longer exists?

Den ons. 1. maj 2019 kl. 22.04 skrev Alexandre Vermeerbergen <
avermeerber...@gmail.com>:

> Hello,
>
> Thanks, and my step issue with Storm 2.0.0 is with some Storm Kafka
> Client breaking changes in KafkaSpoutConfig class.
> See attached source for our homebrewed BasicKafkaSpout, I have at
> least the following compilation errors:
>
>
> ./i3DXSupTopologiesCommons/i3DXSupTopologiesCommons.mj/src/com/dassault_systemes/storm/spout/BasicKafkaSpout.java:58:
> error: cannot find symbol
> logger.debug("topics are: {}", config.getSubscription()
>
>
> ./i3DXSupTopologiesCommons/i3DXSupTopologiesCommons.mj/src/com/dassault_systemes/storm/spout/BasicKafkaSpout.java:77:
> error: cannot find symbol
> if (config.getKeyDeserializer()==null) {
>
>
> ./i3DXSupTopologiesCommons/i3DXSupTopologiesCommons.mj/src/com/dassault_systemes/storm/spout/BasicKafkaSpout.java:91:
> error: cannot find symbol
> this.valueDeserializer =
> config.getValueDeserializer().getClass();
>
> and this more complex error:
>
>
> ./i3DXSupTopologiesCommons/i3DXSupTopologiesCommons.mj/src/com/dassault_systemes/storm/spout/BasicKafkaSpout.java:91:
> error: incompatible types: getClass cannot be converted to Class extends Deserializer>
> this.valueDeserializer =
> config.getValueDeserializer().getClass();
>
>
> What would be the migration path from Storm 1.x to Storm 2.x for this
> storm-kafka-client dependent class?
>
> Kind regards,
> Alexandre Vermeerbergen
>
> Le mer. 1 mai 2019 à 21:18, Stig Rohde Døssing
>  a écrit :
> >
> > Yes, we already have a check that the Maven version is above 3.0.0 (
> > https://github.com/apache/storm/blob/master/pom.xml#L1439), IMO we can
> just
> > bump it to something like 3.5.0. I'll raise an issue for it.
> >
> > Den ons. 1. maj 2019 kl. 20.59 skrev Alexandre Vermeerbergen <
> > avermeerber...@gmail.com>:
> >
> > > Hello Stig,
> > >
> > > Good catch: I upgraded Maven to version 3.6.1 and I rebuilt everything
> > > from source tar.gz without any trouble.
> > >
> > > So I delay my vote and I'm no longer blocked since I have
> > > storm-kafka-client-2.0.0.jar allowing me to rebuilt my topologies.
> > >
> > > Just a question: since my experience showed that build Storm 2.0.0
> > > break with Maven 3.3.9, wouldn't it be possible to improve Storm's
> > > pom.xml to include a Maven version check (see
> > > https://maven.apache.org/enforcer/enforcer-rules/versionRanges.html) ?
> > >
> > > This would improve Storm users' experience a little bit, and sounds
> > > very simple to do (not a blocker for 2.0.0 of course).
> > >
> > > How about that?
> > >
> > > Kind regards,
> > > Alexandre Vermeerbergen
> > >
> > > Le mer. 1 mai 2019 à 19:42, Stig Rohde Døssing
> > >  a écrit :
> > > >
> > > > I haven't seen the error you're getting before, but it looks like
> it's a
> > > > Maven issue, not a Storm issue. Could you try updating to the latest
> > > Maven
> > > > version?
> > > >
> > > > Den ons. 1. maj 2019 kl. 19.30 skrev Alexandre Vermeerbergen <
> > > > avermeerber...@gmail.com>:
> > > >
> > > > > Hello,
> > > > >
> > > > > I don't know if this is worth a -1 because maybe I did something
> > > > > wrong, but I got a failure when trying to build binaries from the
> > > > > sources, see attached log (zipped) with 4 errors, captured from the
> > > > > output of:
> > > > >
> > > > > mvn clean install -DskipTests 2>&1 build_Storm_2.0.0_rc7.log
> > > > >
> > > > > more details on my machine:
> > > > >
> > > > > $ cat /etc/redhat-release
> > > > > CentOS release 6.7 (Final)
> > > > >
> > > > > $ mvn -version
> > > > > Apache Maven 3.3.9 (bb52d8502b132ec0a5a3f4c09453c07478323dc5;
> > > > > 2015-11-10T17:41:47+01:00)
> > > > > Maven home: /home/data/ave/maven/src/apache-maven-3.3.9
> > > > > Java version: 1.8.0_66, vendor: Oracle Corporation
> > > > > Java home: /usr/java/jdk1.8.0_66_x86_64/jdk1.8.0_66/jre
> > > > > Default locale: en_US, platform encoding: UTF-8
> > > > > OS name: "linux", version: "2.6.32-573.el6.x86_64", arch: "amd64",
> > > > > family: "unix"
> > > > >
> > > > > I got couple of JAR files built, but I'm missing the
> > > > > storm-kafka-client one, so am blocked because I need this
> "external"
> > > > > lib to test my topologies rebuilt with Storm 2.0.0 RC7
> > > > >
> > > > > Did I missed something that break my Storm 2.0.0 RC7 build?
> 

Re: [VOTE] Release Apache Storm 2.0.0 RC7

2019-05-01 Thread Alexandre Vermeerbergen
Hello,

Thanks, and my step issue with Storm 2.0.0 is with some Storm Kafka
Client breaking changes in KafkaSpoutConfig class.
See attached source for our homebrewed BasicKafkaSpout, I have at
least the following compilation errors:

./i3DXSupTopologiesCommons/i3DXSupTopologiesCommons.mj/src/com/dassault_systemes/storm/spout/BasicKafkaSpout.java:58:
error: cannot find symbol
logger.debug("topics are: {}", config.getSubscription()

./i3DXSupTopologiesCommons/i3DXSupTopologiesCommons.mj/src/com/dassault_systemes/storm/spout/BasicKafkaSpout.java:77:
error: cannot find symbol
if (config.getKeyDeserializer()==null) {

./i3DXSupTopologiesCommons/i3DXSupTopologiesCommons.mj/src/com/dassault_systemes/storm/spout/BasicKafkaSpout.java:91:
error: cannot find symbol
this.valueDeserializer =
config.getValueDeserializer().getClass();

and this more complex error:

./i3DXSupTopologiesCommons/i3DXSupTopologiesCommons.mj/src/com/dassault_systemes/storm/spout/BasicKafkaSpout.java:91:
error: incompatible types: getClass cannot be converted to Class
this.valueDeserializer =
config.getValueDeserializer().getClass();


What would be the migration path from Storm 1.x to Storm 2.x for this
storm-kafka-client dependent class?

Kind regards,
Alexandre Vermeerbergen

Le mer. 1 mai 2019 à 21:18, Stig Rohde Døssing
 a écrit :
>
> Yes, we already have a check that the Maven version is above 3.0.0 (
> https://github.com/apache/storm/blob/master/pom.xml#L1439), IMO we can just
> bump it to something like 3.5.0. I'll raise an issue for it.
>
> Den ons. 1. maj 2019 kl. 20.59 skrev Alexandre Vermeerbergen <
> avermeerber...@gmail.com>:
>
> > Hello Stig,
> >
> > Good catch: I upgraded Maven to version 3.6.1 and I rebuilt everything
> > from source tar.gz without any trouble.
> >
> > So I delay my vote and I'm no longer blocked since I have
> > storm-kafka-client-2.0.0.jar allowing me to rebuilt my topologies.
> >
> > Just a question: since my experience showed that build Storm 2.0.0
> > break with Maven 3.3.9, wouldn't it be possible to improve Storm's
> > pom.xml to include a Maven version check (see
> > https://maven.apache.org/enforcer/enforcer-rules/versionRanges.html) ?
> >
> > This would improve Storm users' experience a little bit, and sounds
> > very simple to do (not a blocker for 2.0.0 of course).
> >
> > How about that?
> >
> > Kind regards,
> > Alexandre Vermeerbergen
> >
> > Le mer. 1 mai 2019 à 19:42, Stig Rohde Døssing
> >  a écrit :
> > >
> > > I haven't seen the error you're getting before, but it looks like it's a
> > > Maven issue, not a Storm issue. Could you try updating to the latest
> > Maven
> > > version?
> > >
> > > Den ons. 1. maj 2019 kl. 19.30 skrev Alexandre Vermeerbergen <
> > > avermeerber...@gmail.com>:
> > >
> > > > Hello,
> > > >
> > > > I don't know if this is worth a -1 because maybe I did something
> > > > wrong, but I got a failure when trying to build binaries from the
> > > > sources, see attached log (zipped) with 4 errors, captured from the
> > > > output of:
> > > >
> > > > mvn clean install -DskipTests 2>&1 build_Storm_2.0.0_rc7.log
> > > >
> > > > more details on my machine:
> > > >
> > > > $ cat /etc/redhat-release
> > > > CentOS release 6.7 (Final)
> > > >
> > > > $ mvn -version
> > > > Apache Maven 3.3.9 (bb52d8502b132ec0a5a3f4c09453c07478323dc5;
> > > > 2015-11-10T17:41:47+01:00)
> > > > Maven home: /home/data/ave/maven/src/apache-maven-3.3.9
> > > > Java version: 1.8.0_66, vendor: Oracle Corporation
> > > > Java home: /usr/java/jdk1.8.0_66_x86_64/jdk1.8.0_66/jre
> > > > Default locale: en_US, platform encoding: UTF-8
> > > > OS name: "linux", version: "2.6.32-573.el6.x86_64", arch: "amd64",
> > > > family: "unix"
> > > >
> > > > I got couple of JAR files built, but I'm missing the
> > > > storm-kafka-client one, so am blocked because I need this "external"
> > > > lib to test my topologies rebuilt with Storm 2.0.0 RC7
> > > >
> > > > Did I missed something that break my Storm 2.0.0 RC7 build?
> > > >
> > > > Kind regards,
> > > > Alexandre Vermeerbergen
> > > >
> > > >
> > > > Le mar. 30 avr. 2019 à 00:49, P. Taylor Goetz  a
> > écrit
> > > > :
> > > > >
> > > > > This is a call to vote on releasing Apache Storm 2.0.0 (rc7)
> > > > >
> > > > > Full list of changes in this release:
> > > > >
> > > > >
> > > >
> > https://dist.apache.org/repos/dist/dev/storm/apache-storm-2.0.0-rc7/RELEASE_NOTES.html
> > > > >
> > > > > The tag/commit to be voted upon is v2.0.0:
> > > > >
> > > > >
> > > >
> > https://git-wip-us.apache.org/repos/asf?p=storm.git;a=tree;h=007863edd95e838b3df414928c6fa3f28244ab49;hb=2ba95bbd1c911d4fc6363b1c4b9c4c6d86ac9aae
> > > > >
> > > > > The source archive being voted upon can be found here:
> > > > >
> > > > >
> > > >
> > https://dist.apache.org/repos/dist/dev/storm/apache-storm-2.0.0-rc7/apache-storm-2.0.0-src.tar.gz
> > > > >
> > > > > Other release files, signatures and digests can be found 

Re: [VOTE] Release Apache Storm 2.0.0 RC7

2019-05-01 Thread Derek Dagit
> we can just bump it to something like 3.5.0. I'll raise an issue for it.

+1 and Thanks Alexandre for finding this, it is a help.

On Wed, May 1, 2019 at 2:18 PM Stig Rohde Døssing 
wrote:

> Yes, we already have a check that the Maven version is above 3.0.0 (
> https://github.com/apache/storm/blob/master/pom.xml#L1439), IMO we can
> just
> bump it to something like 3.5.0. I'll raise an issue for it.
>
> Den ons. 1. maj 2019 kl. 20.59 skrev Alexandre Vermeerbergen <
> avermeerber...@gmail.com>:
>
> > Hello Stig,
> >
> > Good catch: I upgraded Maven to version 3.6.1 and I rebuilt everything
> > from source tar.gz without any trouble.
> >
> > So I delay my vote and I'm no longer blocked since I have
> > storm-kafka-client-2.0.0.jar allowing me to rebuilt my topologies.
> >
> > Just a question: since my experience showed that build Storm 2.0.0
> > break with Maven 3.3.9, wouldn't it be possible to improve Storm's
> > pom.xml to include a Maven version check (see
> > https://maven.apache.org/enforcer/enforcer-rules/versionRanges.html) ?
> >
> > This would improve Storm users' experience a little bit, and sounds
> > very simple to do (not a blocker for 2.0.0 of course).
> >
> > How about that?
> >
> > Kind regards,
> > Alexandre Vermeerbergen
> >
> > Le mer. 1 mai 2019 à 19:42, Stig Rohde Døssing
> >  a écrit :
> > >
> > > I haven't seen the error you're getting before, but it looks like it's
> a
> > > Maven issue, not a Storm issue. Could you try updating to the latest
> > Maven
> > > version?
> > >
> > > Den ons. 1. maj 2019 kl. 19.30 skrev Alexandre Vermeerbergen <
> > > avermeerber...@gmail.com>:
> > >
> > > > Hello,
> > > >
> > > > I don't know if this is worth a -1 because maybe I did something
> > > > wrong, but I got a failure when trying to build binaries from the
> > > > sources, see attached log (zipped) with 4 errors, captured from the
> > > > output of:
> > > >
> > > > mvn clean install -DskipTests 2>&1 build_Storm_2.0.0_rc7.log
> > > >
> > > > more details on my machine:
> > > >
> > > > $ cat /etc/redhat-release
> > > > CentOS release 6.7 (Final)
> > > >
> > > > $ mvn -version
> > > > Apache Maven 3.3.9 (bb52d8502b132ec0a5a3f4c09453c07478323dc5;
> > > > 2015-11-10T17:41:47+01:00)
> > > > Maven home: /home/data/ave/maven/src/apache-maven-3.3.9
> > > > Java version: 1.8.0_66, vendor: Oracle Corporation
> > > > Java home: /usr/java/jdk1.8.0_66_x86_64/jdk1.8.0_66/jre
> > > > Default locale: en_US, platform encoding: UTF-8
> > > > OS name: "linux", version: "2.6.32-573.el6.x86_64", arch: "amd64",
> > > > family: "unix"
> > > >
> > > > I got couple of JAR files built, but I'm missing the
> > > > storm-kafka-client one, so am blocked because I need this "external"
> > > > lib to test my topologies rebuilt with Storm 2.0.0 RC7
> > > >
> > > > Did I missed something that break my Storm 2.0.0 RC7 build?
> > > >
> > > > Kind regards,
> > > > Alexandre Vermeerbergen
> > > >
> > > >
> > > > Le mar. 30 avr. 2019 à 00:49, P. Taylor Goetz  a
> > écrit
> > > > :
> > > > >
> > > > > This is a call to vote on releasing Apache Storm 2.0.0 (rc7)
> > > > >
> > > > > Full list of changes in this release:
> > > > >
> > > > >
> > > >
> >
> https://dist.apache.org/repos/dist/dev/storm/apache-storm-2.0.0-rc7/RELEASE_NOTES.html
> > > > >
> > > > > The tag/commit to be voted upon is v2.0.0:
> > > > >
> > > > >
> > > >
> >
> https://git-wip-us.apache.org/repos/asf?p=storm.git;a=tree;h=007863edd95e838b3df414928c6fa3f28244ab49;hb=2ba95bbd1c911d4fc6363b1c4b9c4c6d86ac9aae
> > > > >
> > > > > The source archive being voted upon can be found here:
> > > > >
> > > > >
> > > >
> >
> https://dist.apache.org/repos/dist/dev/storm/apache-storm-2.0.0-rc7/apache-storm-2.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-2.0.0-rc7/
> > > > >
> > > > > 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-1079
> > > > >
> > > > > Please vote on releasing this package as Apache Storm 2.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 2.0.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 2.0.0 RC7

2019-05-01 Thread Stig Rohde Døssing
Yes, we already have a check that the Maven version is above 3.0.0 (
https://github.com/apache/storm/blob/master/pom.xml#L1439), IMO we can just
bump it to something like 3.5.0. I'll raise an issue for it.

Den ons. 1. maj 2019 kl. 20.59 skrev Alexandre Vermeerbergen <
avermeerber...@gmail.com>:

> Hello Stig,
>
> Good catch: I upgraded Maven to version 3.6.1 and I rebuilt everything
> from source tar.gz without any trouble.
>
> So I delay my vote and I'm no longer blocked since I have
> storm-kafka-client-2.0.0.jar allowing me to rebuilt my topologies.
>
> Just a question: since my experience showed that build Storm 2.0.0
> break with Maven 3.3.9, wouldn't it be possible to improve Storm's
> pom.xml to include a Maven version check (see
> https://maven.apache.org/enforcer/enforcer-rules/versionRanges.html) ?
>
> This would improve Storm users' experience a little bit, and sounds
> very simple to do (not a blocker for 2.0.0 of course).
>
> How about that?
>
> Kind regards,
> Alexandre Vermeerbergen
>
> Le mer. 1 mai 2019 à 19:42, Stig Rohde Døssing
>  a écrit :
> >
> > I haven't seen the error you're getting before, but it looks like it's a
> > Maven issue, not a Storm issue. Could you try updating to the latest
> Maven
> > version?
> >
> > Den ons. 1. maj 2019 kl. 19.30 skrev Alexandre Vermeerbergen <
> > avermeerber...@gmail.com>:
> >
> > > Hello,
> > >
> > > I don't know if this is worth a -1 because maybe I did something
> > > wrong, but I got a failure when trying to build binaries from the
> > > sources, see attached log (zipped) with 4 errors, captured from the
> > > output of:
> > >
> > > mvn clean install -DskipTests 2>&1 build_Storm_2.0.0_rc7.log
> > >
> > > more details on my machine:
> > >
> > > $ cat /etc/redhat-release
> > > CentOS release 6.7 (Final)
> > >
> > > $ mvn -version
> > > Apache Maven 3.3.9 (bb52d8502b132ec0a5a3f4c09453c07478323dc5;
> > > 2015-11-10T17:41:47+01:00)
> > > Maven home: /home/data/ave/maven/src/apache-maven-3.3.9
> > > Java version: 1.8.0_66, vendor: Oracle Corporation
> > > Java home: /usr/java/jdk1.8.0_66_x86_64/jdk1.8.0_66/jre
> > > Default locale: en_US, platform encoding: UTF-8
> > > OS name: "linux", version: "2.6.32-573.el6.x86_64", arch: "amd64",
> > > family: "unix"
> > >
> > > I got couple of JAR files built, but I'm missing the
> > > storm-kafka-client one, so am blocked because I need this "external"
> > > lib to test my topologies rebuilt with Storm 2.0.0 RC7
> > >
> > > Did I missed something that break my Storm 2.0.0 RC7 build?
> > >
> > > Kind regards,
> > > Alexandre Vermeerbergen
> > >
> > >
> > > Le mar. 30 avr. 2019 à 00:49, P. Taylor Goetz  a
> écrit
> > > :
> > > >
> > > > This is a call to vote on releasing Apache Storm 2.0.0 (rc7)
> > > >
> > > > Full list of changes in this release:
> > > >
> > > >
> > >
> https://dist.apache.org/repos/dist/dev/storm/apache-storm-2.0.0-rc7/RELEASE_NOTES.html
> > > >
> > > > The tag/commit to be voted upon is v2.0.0:
> > > >
> > > >
> > >
> https://git-wip-us.apache.org/repos/asf?p=storm.git;a=tree;h=007863edd95e838b3df414928c6fa3f28244ab49;hb=2ba95bbd1c911d4fc6363b1c4b9c4c6d86ac9aae
> > > >
> > > > The source archive being voted upon can be found here:
> > > >
> > > >
> > >
> https://dist.apache.org/repos/dist/dev/storm/apache-storm-2.0.0-rc7/apache-storm-2.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-2.0.0-rc7/
> > > >
> > > > 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-1079
> > > >
> > > > Please vote on releasing this package as Apache Storm 2.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 2.0.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 2.0.0 RC7

2019-05-01 Thread Alexandre Vermeerbergen
Hello Stig,

Good catch: I upgraded Maven to version 3.6.1 and I rebuilt everything
from source tar.gz without any trouble.

So I delay my vote and I'm no longer blocked since I have
storm-kafka-client-2.0.0.jar allowing me to rebuilt my topologies.

Just a question: since my experience showed that build Storm 2.0.0
break with Maven 3.3.9, wouldn't it be possible to improve Storm's
pom.xml to include a Maven version check (see
https://maven.apache.org/enforcer/enforcer-rules/versionRanges.html) ?

This would improve Storm users' experience a little bit, and sounds
very simple to do (not a blocker for 2.0.0 of course).

How about that?

Kind regards,
Alexandre Vermeerbergen

Le mer. 1 mai 2019 à 19:42, Stig Rohde Døssing
 a écrit :
>
> I haven't seen the error you're getting before, but it looks like it's a
> Maven issue, not a Storm issue. Could you try updating to the latest Maven
> version?
>
> Den ons. 1. maj 2019 kl. 19.30 skrev Alexandre Vermeerbergen <
> avermeerber...@gmail.com>:
>
> > Hello,
> >
> > I don't know if this is worth a -1 because maybe I did something
> > wrong, but I got a failure when trying to build binaries from the
> > sources, see attached log (zipped) with 4 errors, captured from the
> > output of:
> >
> > mvn clean install -DskipTests 2>&1 build_Storm_2.0.0_rc7.log
> >
> > more details on my machine:
> >
> > $ cat /etc/redhat-release
> > CentOS release 6.7 (Final)
> >
> > $ mvn -version
> > Apache Maven 3.3.9 (bb52d8502b132ec0a5a3f4c09453c07478323dc5;
> > 2015-11-10T17:41:47+01:00)
> > Maven home: /home/data/ave/maven/src/apache-maven-3.3.9
> > Java version: 1.8.0_66, vendor: Oracle Corporation
> > Java home: /usr/java/jdk1.8.0_66_x86_64/jdk1.8.0_66/jre
> > Default locale: en_US, platform encoding: UTF-8
> > OS name: "linux", version: "2.6.32-573.el6.x86_64", arch: "amd64",
> > family: "unix"
> >
> > I got couple of JAR files built, but I'm missing the
> > storm-kafka-client one, so am blocked because I need this "external"
> > lib to test my topologies rebuilt with Storm 2.0.0 RC7
> >
> > Did I missed something that break my Storm 2.0.0 RC7 build?
> >
> > Kind regards,
> > Alexandre Vermeerbergen
> >
> >
> > Le mar. 30 avr. 2019 à 00:49, P. Taylor Goetz  a écrit
> > :
> > >
> > > This is a call to vote on releasing Apache Storm 2.0.0 (rc7)
> > >
> > > Full list of changes in this release:
> > >
> > >
> > https://dist.apache.org/repos/dist/dev/storm/apache-storm-2.0.0-rc7/RELEASE_NOTES.html
> > >
> > > The tag/commit to be voted upon is v2.0.0:
> > >
> > >
> > https://git-wip-us.apache.org/repos/asf?p=storm.git;a=tree;h=007863edd95e838b3df414928c6fa3f28244ab49;hb=2ba95bbd1c911d4fc6363b1c4b9c4c6d86ac9aae
> > >
> > > The source archive being voted upon can be found here:
> > >
> > >
> > https://dist.apache.org/repos/dist/dev/storm/apache-storm-2.0.0-rc7/apache-storm-2.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-2.0.0-rc7/
> > >
> > > 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-1079
> > >
> > > Please vote on releasing this package as Apache Storm 2.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 2.0.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 2.0.0 RC7

2019-05-01 Thread Stig Rohde Døssing
I haven't seen the error you're getting before, but it looks like it's a
Maven issue, not a Storm issue. Could you try updating to the latest Maven
version?

Den ons. 1. maj 2019 kl. 19.30 skrev Alexandre Vermeerbergen <
avermeerber...@gmail.com>:

> Hello,
>
> I don't know if this is worth a -1 because maybe I did something
> wrong, but I got a failure when trying to build binaries from the
> sources, see attached log (zipped) with 4 errors, captured from the
> output of:
>
> mvn clean install -DskipTests 2>&1 build_Storm_2.0.0_rc7.log
>
> more details on my machine:
>
> $ cat /etc/redhat-release
> CentOS release 6.7 (Final)
>
> $ mvn -version
> Apache Maven 3.3.9 (bb52d8502b132ec0a5a3f4c09453c07478323dc5;
> 2015-11-10T17:41:47+01:00)
> Maven home: /home/data/ave/maven/src/apache-maven-3.3.9
> Java version: 1.8.0_66, vendor: Oracle Corporation
> Java home: /usr/java/jdk1.8.0_66_x86_64/jdk1.8.0_66/jre
> Default locale: en_US, platform encoding: UTF-8
> OS name: "linux", version: "2.6.32-573.el6.x86_64", arch: "amd64",
> family: "unix"
>
> I got couple of JAR files built, but I'm missing the
> storm-kafka-client one, so am blocked because I need this "external"
> lib to test my topologies rebuilt with Storm 2.0.0 RC7
>
> Did I missed something that break my Storm 2.0.0 RC7 build?
>
> Kind regards,
> Alexandre Vermeerbergen
>
>
> Le mar. 30 avr. 2019 à 00:49, P. Taylor Goetz  a écrit
> :
> >
> > This is a call to vote on releasing Apache Storm 2.0.0 (rc7)
> >
> > Full list of changes in this release:
> >
> >
> https://dist.apache.org/repos/dist/dev/storm/apache-storm-2.0.0-rc7/RELEASE_NOTES.html
> >
> > The tag/commit to be voted upon is v2.0.0:
> >
> >
> https://git-wip-us.apache.org/repos/asf?p=storm.git;a=tree;h=007863edd95e838b3df414928c6fa3f28244ab49;hb=2ba95bbd1c911d4fc6363b1c4b9c4c6d86ac9aae
> >
> > The source archive being voted upon can be found here:
> >
> >
> https://dist.apache.org/repos/dist/dev/storm/apache-storm-2.0.0-rc7/apache-storm-2.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-2.0.0-rc7/
> >
> > 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-1079
> >
> > Please vote on releasing this package as Apache Storm 2.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 2.0.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 2.0.0 RC7

2019-05-01 Thread Stig Rohde Døssing
Alexandre,

I think we tend to assume people use a dependency management tool:
https://storm.apache.org/releases/2.0.0-SNAPSHOT/Maven.html

The easiest way for you to get a list of the jars that need to be on the
classpath is to make a dummy Maven project, add the snippet I linked to the
POM, and then use the dependency plugin to get a list of dependencies (do
"mvn dependency:list" in the project root). You could also run that command
from the storm-client directory in the Storm source.

Regarding which dependencies you should have, the link snippet is enough
for a production jar. If you are running tests using a LocalCluster, you
will also need to have storm-server on the test classpath.

Neither jar should be included in your fat jar. Storm puts them on the
classpath when you deploy to a cluster.

Den ons. 1. maj 2019 kl. 09.30 skrev Alexandre Vermeerbergen <
avermeerber...@gmail.com>:

> Hello,
>
> I tried to rebuild my topologies with 2.0, but I hit a first trivial
> issue: I used to rely on storm-core-.jar in our classpath to
> resolve Storm API dependencies, but storm-core-2.0.0.jar doesn't seem
> to contain anymore classes like org.apache.storm.task.OutputCollector
> which is used in our topologies' code.
>
> I did a "grep org.apache.storm.task.OutputCollector  directory>" and found that this kind of classes are now in
> storm-client-2.0.0.jar
>
> I'm OK to change my dependencies, but then, to avoid other people
> falling into this trap, couldn't Storm 2.0.0 documentation tell which
> are the build-time dependencies?
>
> I also need to know whether or not my BigJars must embed
> storm-client.2.0.0.jar (we use to embed storm-core-.jar in
> our BigJar)
>
> Kind regards,
> Alexandre Vermeerbergen
>
> Le mar. 30 avr. 2019 à 23:49, Roshan Naik
>  a écrit :
> >
> >  Yes, you need to rebuild your topology jars against 2.0.
> > If you have some settings to tweak perf with 1.x,  refer to
> https://github.com/apache/storm/blob/master/docs/Performance.md for the
> 2.x configs.
> >
> > On Tuesday, April 30, 2019, 2:41:49 PM PDT, Alexandre Vermeerbergen <
> avermeerber...@gmail.com> wrote:
> >
> >  Hello,
> >
> > I'm eager to test Storm 2.0.0 with my complex topologies, but first of
> > all: do I need to rebuild all my topologies' Big Jars with Storm
> > 2.0.0, or may I try my existing Storm 1.2.3 (recent snapshot)-based
> > Big Jars ?
> >
> > Kind regards,
> > Alexandre Vermeerbergen
> >
> > Le mar. 30 avr. 2019 à 00:49, P. Taylor Goetz  a
> écrit :
> > >
> > > This is a call to vote on releasing Apache Storm 2.0.0 (rc7)
> > >
> > > Full list of changes in this release:
> > >
> > >
> https://dist.apache.org/repos/dist/dev/storm/apache-storm-2.0.0-rc7/RELEASE_NOTES.html
> > >
> > > The tag/commit to be voted upon is v2.0.0:
> > >
> > >
> https://git-wip-us.apache.org/repos/asf?p=storm.git;a=tree;h=007863edd95e838b3df414928c6fa3f28244ab49;hb=2ba95bbd1c911d4fc6363b1c4b9c4c6d86ac9aae
> > >
> > > The source archive being voted upon can be found here:
> > >
> > >
> https://dist.apache.org/repos/dist/dev/storm/apache-storm-2.0.0-rc7/apache-storm-2.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-2.0.0-rc7/
> > >
> > > 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-1079
> > >
> > > Please vote on releasing this package as Apache Storm 2.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 2.0.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 2.0.0 RC7

2019-05-01 Thread Alexandre Vermeerbergen
Hello,

I tried to rebuild my topologies with 2.0, but I hit a first trivial
issue: I used to rely on storm-core-.jar in our classpath to
resolve Storm API dependencies, but storm-core-2.0.0.jar doesn't seem
to contain anymore classes like org.apache.storm.task.OutputCollector
which is used in our topologies' code.

I did a "grep org.apache.storm.task.OutputCollector " and found that this kind of classes are now in
storm-client-2.0.0.jar

I'm OK to change my dependencies, but then, to avoid other people
falling into this trap, couldn't Storm 2.0.0 documentation tell which
are the build-time dependencies?

I also need to know whether or not my BigJars must embed
storm-client.2.0.0.jar (we use to embed storm-core-.jar in
our BigJar)

Kind regards,
Alexandre Vermeerbergen

Le mar. 30 avr. 2019 à 23:49, Roshan Naik
 a écrit :
>
>  Yes, you need to rebuild your topology jars against 2.0.
> If you have some settings to tweak perf with 1.x,  refer to 
> https://github.com/apache/storm/blob/master/docs/Performance.md for the 2.x 
> configs.
>
> On Tuesday, April 30, 2019, 2:41:49 PM PDT, Alexandre Vermeerbergen 
>  wrote:
>
>  Hello,
>
> I'm eager to test Storm 2.0.0 with my complex topologies, but first of
> all: do I need to rebuild all my topologies' Big Jars with Storm
> 2.0.0, or may I try my existing Storm 1.2.3 (recent snapshot)-based
> Big Jars ?
>
> Kind regards,
> Alexandre Vermeerbergen
>
> Le mar. 30 avr. 2019 à 00:49, P. Taylor Goetz  a écrit :
> >
> > This is a call to vote on releasing Apache Storm 2.0.0 (rc7)
> >
> > Full list of changes in this release:
> >
> > https://dist.apache.org/repos/dist/dev/storm/apache-storm-2.0.0-rc7/RELEASE_NOTES.html
> >
> > The tag/commit to be voted upon is v2.0.0:
> >
> > https://git-wip-us.apache.org/repos/asf?p=storm.git;a=tree;h=007863edd95e838b3df414928c6fa3f28244ab49;hb=2ba95bbd1c911d4fc6363b1c4b9c4c6d86ac9aae
> >
> > The source archive being voted upon can be found here:
> >
> > https://dist.apache.org/repos/dist/dev/storm/apache-storm-2.0.0-rc7/apache-storm-2.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-2.0.0-rc7/
> >
> > 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-1079
> >
> > Please vote on releasing this package as Apache Storm 2.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 2.0.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 2.0.0 RC7

2019-04-30 Thread Arun Mahadevan
+1
- Downloaded binaries
- Brought up a local cluster
- Ran a few topologies and checked the output and logs.

If I just click on the "Owner" link from the UI, it displays a red banner
with a warning.
"This user's topologies are in danger of being unscheduled due to the
owner's over-use of cluster resources.
Please keep this user's resource consumption within guaranteed bounds to
ensure topologies for this user will continue to run."

It just looks odd and I am not sure whats the criteria to decide if there
is overuse. Looks like its displayed irrespective of the topology.

Anyways I don't think its a blocker, something we could fix in the next
release.

Thanks,
Arun



On Tue, 30 Apr 2019 at 14:49, Roshan Naik 
wrote:

>  Yes, you need to rebuild your topology jars against 2.0.
> If you have some settings to tweak perf with 1.x,  refer to
> https://github.com/apache/storm/blob/master/docs/Performance.md for the
> 2.x configs.
>
> On Tuesday, April 30, 2019, 2:41:49 PM PDT, Alexandre Vermeerbergen <
> avermeerber...@gmail.com> wrote:
>
>  Hello,
>
> I'm eager to test Storm 2.0.0 with my complex topologies, but first of
> all: do I need to rebuild all my topologies' Big Jars with Storm
> 2.0.0, or may I try my existing Storm 1.2.3 (recent snapshot)-based
> Big Jars ?
>
> Kind regards,
> Alexandre Vermeerbergen
>
> Le mar. 30 avr. 2019 à 00:49, P. Taylor Goetz  a écrit
> :
> >
> > This is a call to vote on releasing Apache Storm 2.0.0 (rc7)
> >
> > Full list of changes in this release:
> >
> >
> https://dist.apache.org/repos/dist/dev/storm/apache-storm-2.0.0-rc7/RELEASE_NOTES.html
> >
> > The tag/commit to be voted upon is v2.0.0:
> >
> >
> https://git-wip-us.apache.org/repos/asf?p=storm.git;a=tree;h=007863edd95e838b3df414928c6fa3f28244ab49;hb=2ba95bbd1c911d4fc6363b1c4b9c4c6d86ac9aae
> >
> > The source archive being voted upon can be found here:
> >
> >
> https://dist.apache.org/repos/dist/dev/storm/apache-storm-2.0.0-rc7/apache-storm-2.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-2.0.0-rc7/
> >
> > 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-1079
> >
> > Please vote on releasing this package as Apache Storm 2.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 2.0.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 2.0.0 RC7

2019-04-30 Thread Roshan Naik
 Yes, you need to rebuild your topology jars against 2.0. 
If you have some settings to tweak perf with 1.x,  refer to 
https://github.com/apache/storm/blob/master/docs/Performance.md for the 2.x 
configs.

On Tuesday, April 30, 2019, 2:41:49 PM PDT, Alexandre Vermeerbergen 
 wrote:  
 
 Hello,

I'm eager to test Storm 2.0.0 with my complex topologies, but first of
all: do I need to rebuild all my topologies' Big Jars with Storm
2.0.0, or may I try my existing Storm 1.2.3 (recent snapshot)-based
Big Jars ?

Kind regards,
Alexandre Vermeerbergen

Le mar. 30 avr. 2019 à 00:49, P. Taylor Goetz  a écrit :
>
> This is a call to vote on releasing Apache Storm 2.0.0 (rc7)
>
> Full list of changes in this release:
>
> https://dist.apache.org/repos/dist/dev/storm/apache-storm-2.0.0-rc7/RELEASE_NOTES.html
>
> The tag/commit to be voted upon is v2.0.0:
>
> https://git-wip-us.apache.org/repos/asf?p=storm.git;a=tree;h=007863edd95e838b3df414928c6fa3f28244ab49;hb=2ba95bbd1c911d4fc6363b1c4b9c4c6d86ac9aae
>
> The source archive being voted upon can be found here:
>
> https://dist.apache.org/repos/dist/dev/storm/apache-storm-2.0.0-rc7/apache-storm-2.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-2.0.0-rc7/
>
> 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-1079
>
> Please vote on releasing this package as Apache Storm 2.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 2.0.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 2.0.0 RC7

2019-04-30 Thread Alexandre Vermeerbergen
Hello,

I'm eager to test Storm 2.0.0 with my complex topologies, but first of
all: do I need to rebuild all my topologies' Big Jars with Storm
2.0.0, or may I try my existing Storm 1.2.3 (recent snapshot)-based
Big Jars ?

Kind regards,
Alexandre Vermeerbergen

Le mar. 30 avr. 2019 à 00:49, P. Taylor Goetz  a écrit :
>
> This is a call to vote on releasing Apache Storm 2.0.0 (rc7)
>
> Full list of changes in this release:
>
> https://dist.apache.org/repos/dist/dev/storm/apache-storm-2.0.0-rc7/RELEASE_NOTES.html
>
> The tag/commit to be voted upon is v2.0.0:
>
> https://git-wip-us.apache.org/repos/asf?p=storm.git;a=tree;h=007863edd95e838b3df414928c6fa3f28244ab49;hb=2ba95bbd1c911d4fc6363b1c4b9c4c6d86ac9aae
>
> The source archive being voted upon can be found here:
>
> https://dist.apache.org/repos/dist/dev/storm/apache-storm-2.0.0-rc7/apache-storm-2.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-2.0.0-rc7/
>
> 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-1079
>
> Please vote on releasing this package as Apache Storm 2.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 2.0.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 2.0.0 RC7

2019-04-30 Thread Jungtaek Lim
+1 (binding)

> source

- verify file (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 8
-- source, tar.gz : OK

- build source dist
-- source, tar.gz : OK

- build binary dist
-- source, tar.gz : OK

> binary

- verify file (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

- launch daemons : OK

- run ExclamationTopology (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

Thanks,
Jungtaek Lim (HeartSaVioR)

On Wed, May 1, 2019 at 4:38 AM Kishorkumar Patil
 wrote:

> Built from the source code. All Unit tests passed.
> Ran some basic tests with WordCountTopology, BasicDRPCTopology.
> Run ThroughputVsLatency for comparison and it looks good- with 99.9% of
> events processed within 5.4ms.
> +1
>
>
> On Tue, Apr 30, 2019 at 2:16 PM Derek Dagit 
> wrote:
>
> > OK ran the distribution from the tarball, and built and ran from the src
> > tarball on macOS with JDK8. Tested WordCountTopology on both and it
> seemed
> > fine.
> >
> > +1
> >
> > On Tue, Apr 30, 2019 at 11:55 AM Derek Dagit  wrote:
> >
> > > Sent too soon:
> > >
> > > - I verified sigs and sums
> > > - Built the source from the tarball on macOS
> > >   - I saw a couple of test errors because the local stormconf.ser was
> > gone
> > > when the supervisor tried to open it, and this resulted in a crash. I
> > > re-ran the tests and they passed.
> > >   - I do not think this is a blocker, but it is probable worth looking
> > > into.
> > >
> > > I'll continue to evaluate the RC, but wanted to send this out in case
> > > anyone else has seen something similar.
> > >
> > > On Tue, Apr 30, 2019 at 11:52 AM Derek Dagit  wrote:
> > >
> > >> - I verified sigs and sums
> > >>
> > >> On Mon, Apr 29, 2019 at 5:49 PM P. Taylor Goetz 
> > >> wrote:
> > >>
> > >>> This is a call to vote on releasing Apache Storm 2.0.0 (rc7)
> > >>>
> > >>> Full list of changes in this release:
> > >>>
> > >>>
> > >>>
> >
> https://dist.apache.org/repos/dist/dev/storm/apache-storm-2.0.0-rc7/RELEASE_NOTES.html
> > >>>
> > >>> The tag/commit to be voted upon is v2.0.0:
> > >>>
> > >>>
> > >>>
> >
> https://git-wip-us.apache.org/repos/asf?p=storm.git;a=tree;h=007863edd95e838b3df414928c6fa3f28244ab49;hb=2ba95bbd1c911d4fc6363b1c4b9c4c6d86ac9aae
> > >>>
> > >>> The source archive being voted upon can be found here:
> > >>>
> > >>>
> > >>>
> >
> https://dist.apache.org/repos/dist/dev/storm/apache-storm-2.0.0-rc7/apache-storm-2.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-2.0.0-rc7/
> > >>>
> > >>> 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-1079
> > >>>
> > >>> Please vote on releasing this package as Apache Storm 2.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 2.0.0
> > >>> [ ]  0 No opinion
> > >>> [ ] -1 Do not release this package because...
> > >>>
> > >>> Thanks to everyone who contributed to this release.
> > >>>
> > >>> -Taylor
> > >>
> > >>
> >
>


-- 
Name : Jungtaek Lim
Blog : http://medium.com/@heartsavior
Twitter : http://twitter.com/heartsavior
LinkedIn : http://www.linkedin.com/in/heartsavior


Re: [VOTE] Release Apache Storm 2.0.0 RC7

2019-04-30 Thread Kishorkumar Patil
Built from the source code. All Unit tests passed.
Ran some basic tests with WordCountTopology, BasicDRPCTopology.
Run ThroughputVsLatency for comparison and it looks good- with 99.9% of
events processed within 5.4ms.
+1


On Tue, Apr 30, 2019 at 2:16 PM Derek Dagit  wrote:

> OK ran the distribution from the tarball, and built and ran from the src
> tarball on macOS with JDK8. Tested WordCountTopology on both and it seemed
> fine.
>
> +1
>
> On Tue, Apr 30, 2019 at 11:55 AM Derek Dagit  wrote:
>
> > Sent too soon:
> >
> > - I verified sigs and sums
> > - Built the source from the tarball on macOS
> >   - I saw a couple of test errors because the local stormconf.ser was
> gone
> > when the supervisor tried to open it, and this resulted in a crash. I
> > re-ran the tests and they passed.
> >   - I do not think this is a blocker, but it is probable worth looking
> > into.
> >
> > I'll continue to evaluate the RC, but wanted to send this out in case
> > anyone else has seen something similar.
> >
> > On Tue, Apr 30, 2019 at 11:52 AM Derek Dagit  wrote:
> >
> >> - I verified sigs and sums
> >>
> >> On Mon, Apr 29, 2019 at 5:49 PM P. Taylor Goetz 
> >> wrote:
> >>
> >>> This is a call to vote on releasing Apache Storm 2.0.0 (rc7)
> >>>
> >>> Full list of changes in this release:
> >>>
> >>>
> >>>
> https://dist.apache.org/repos/dist/dev/storm/apache-storm-2.0.0-rc7/RELEASE_NOTES.html
> >>>
> >>> The tag/commit to be voted upon is v2.0.0:
> >>>
> >>>
> >>>
> https://git-wip-us.apache.org/repos/asf?p=storm.git;a=tree;h=007863edd95e838b3df414928c6fa3f28244ab49;hb=2ba95bbd1c911d4fc6363b1c4b9c4c6d86ac9aae
> >>>
> >>> The source archive being voted upon can be found here:
> >>>
> >>>
> >>>
> https://dist.apache.org/repos/dist/dev/storm/apache-storm-2.0.0-rc7/apache-storm-2.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-2.0.0-rc7/
> >>>
> >>> 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-1079
> >>>
> >>> Please vote on releasing this package as Apache Storm 2.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 2.0.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 2.0.0 RC7

2019-04-30 Thread Stig Rohde Døssing
+1

Ran tests from source zip
Installed and ran ExclamationTopology on single-node cluster from binary zip
Verified no unexpected error logs.
Clicked around a bit in UI, verified that logviewer works.
Checked out tag and ran the license check plugin to generate
DEPENDENCY-LICENSES. Verified that there are no category X licenses. A
number of dependency versions have changed, but I don't think this should
block the release. I'll see if we can automate generating the license files
a bit more, so we don't forget to update them.

Derek, I'm not sure about a missing stormconf.ser, but there are a couple
of open issues impacting tests stability. See
https://issues.apache.org/jira/browse/STORM-3376 and
https://issues.apache.org/jira/browse/STORM-3379. Maybe one of them fixes
it? I've run the integration test and localcluster tests a bunch of times
with these fixes, and the tests looks stable to me. If you get the errors
occasionally, it would be nice if you could see if they are still there
when applying the two fixes.

Den tir. 30. apr. 2019 kl. 20.16 skrev Derek Dagit :

> OK ran the distribution from the tarball, and built and ran from the src
> tarball on macOS with JDK8. Tested WordCountTopology on both and it seemed
> fine.
>
> +1
>
> On Tue, Apr 30, 2019 at 11:55 AM Derek Dagit  wrote:
>
> > Sent too soon:
> >
> > - I verified sigs and sums
> > - Built the source from the tarball on macOS
> >   - I saw a couple of test errors because the local stormconf.ser was
> gone
> > when the supervisor tried to open it, and this resulted in a crash. I
> > re-ran the tests and they passed.
> >   - I do not think this is a blocker, but it is probable worth looking
> > into.
> >
> > I'll continue to evaluate the RC, but wanted to send this out in case
> > anyone else has seen something similar.
> >
> > On Tue, Apr 30, 2019 at 11:52 AM Derek Dagit  wrote:
> >
> >> - I verified sigs and sums
> >>
> >> On Mon, Apr 29, 2019 at 5:49 PM P. Taylor Goetz 
> >> wrote:
> >>
> >>> This is a call to vote on releasing Apache Storm 2.0.0 (rc7)
> >>>
> >>> Full list of changes in this release:
> >>>
> >>>
> >>>
> https://dist.apache.org/repos/dist/dev/storm/apache-storm-2.0.0-rc7/RELEASE_NOTES.html
> >>>
> >>> The tag/commit to be voted upon is v2.0.0:
> >>>
> >>>
> >>>
> https://git-wip-us.apache.org/repos/asf?p=storm.git;a=tree;h=007863edd95e838b3df414928c6fa3f28244ab49;hb=2ba95bbd1c911d4fc6363b1c4b9c4c6d86ac9aae
> >>>
> >>> The source archive being voted upon can be found here:
> >>>
> >>>
> >>>
> https://dist.apache.org/repos/dist/dev/storm/apache-storm-2.0.0-rc7/apache-storm-2.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-2.0.0-rc7/
> >>>
> >>> 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-1079
> >>>
> >>> Please vote on releasing this package as Apache Storm 2.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 2.0.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 2.0.0 RC7

2019-04-30 Thread Derek Dagit
OK ran the distribution from the tarball, and built and ran from the src
tarball on macOS with JDK8. Tested WordCountTopology on both and it seemed
fine.

+1

On Tue, Apr 30, 2019 at 11:55 AM Derek Dagit  wrote:

> Sent too soon:
>
> - I verified sigs and sums
> - Built the source from the tarball on macOS
>   - I saw a couple of test errors because the local stormconf.ser was gone
> when the supervisor tried to open it, and this resulted in a crash. I
> re-ran the tests and they passed.
>   - I do not think this is a blocker, but it is probable worth looking
> into.
>
> I'll continue to evaluate the RC, but wanted to send this out in case
> anyone else has seen something similar.
>
> On Tue, Apr 30, 2019 at 11:52 AM Derek Dagit  wrote:
>
>> - I verified sigs and sums
>>
>> On Mon, Apr 29, 2019 at 5:49 PM P. Taylor Goetz 
>> wrote:
>>
>>> This is a call to vote on releasing Apache Storm 2.0.0 (rc7)
>>>
>>> Full list of changes in this release:
>>>
>>>
>>> https://dist.apache.org/repos/dist/dev/storm/apache-storm-2.0.0-rc7/RELEASE_NOTES.html
>>>
>>> The tag/commit to be voted upon is v2.0.0:
>>>
>>>
>>> https://git-wip-us.apache.org/repos/asf?p=storm.git;a=tree;h=007863edd95e838b3df414928c6fa3f28244ab49;hb=2ba95bbd1c911d4fc6363b1c4b9c4c6d86ac9aae
>>>
>>> The source archive being voted upon can be found here:
>>>
>>>
>>> https://dist.apache.org/repos/dist/dev/storm/apache-storm-2.0.0-rc7/apache-storm-2.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-2.0.0-rc7/
>>>
>>> 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-1079
>>>
>>> Please vote on releasing this package as Apache Storm 2.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 2.0.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 2.0.0 RC7

2019-04-30 Thread Derek Dagit
Sent too soon:

- I verified sigs and sums
- Built the source from the tarball on macOS
  - I saw a couple of test errors because the local stormconf.ser was gone
when the supervisor tried to open it, and this resulted in a crash. I
re-ran the tests and they passed.
  - I do not think this is a blocker, but it is probable worth looking into.

I'll continue to evaluate the RC, but wanted to send this out in case
anyone else has seen something similar.

On Tue, Apr 30, 2019 at 11:52 AM Derek Dagit  wrote:

> - I verified sigs and sums
>
> On Mon, Apr 29, 2019 at 5:49 PM P. Taylor Goetz  wrote:
>
>> This is a call to vote on releasing Apache Storm 2.0.0 (rc7)
>>
>> Full list of changes in this release:
>>
>>
>> https://dist.apache.org/repos/dist/dev/storm/apache-storm-2.0.0-rc7/RELEASE_NOTES.html
>>
>> The tag/commit to be voted upon is v2.0.0:
>>
>>
>> https://git-wip-us.apache.org/repos/asf?p=storm.git;a=tree;h=007863edd95e838b3df414928c6fa3f28244ab49;hb=2ba95bbd1c911d4fc6363b1c4b9c4c6d86ac9aae
>>
>> The source archive being voted upon can be found here:
>>
>>
>> https://dist.apache.org/repos/dist/dev/storm/apache-storm-2.0.0-rc7/apache-storm-2.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-2.0.0-rc7/
>>
>> 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-1079
>>
>> Please vote on releasing this package as Apache Storm 2.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 2.0.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 2.0.0 RC7

2019-04-30 Thread Derek Dagit
- I verified sigs and sums

On Mon, Apr 29, 2019 at 5:49 PM P. Taylor Goetz  wrote:

> This is a call to vote on releasing Apache Storm 2.0.0 (rc7)
>
> Full list of changes in this release:
>
>
> https://dist.apache.org/repos/dist/dev/storm/apache-storm-2.0.0-rc7/RELEASE_NOTES.html
>
> The tag/commit to be voted upon is v2.0.0:
>
>
> https://git-wip-us.apache.org/repos/asf?p=storm.git;a=tree;h=007863edd95e838b3df414928c6fa3f28244ab49;hb=2ba95bbd1c911d4fc6363b1c4b9c4c6d86ac9aae
>
> The source archive being voted upon can be found here:
>
>
> https://dist.apache.org/repos/dist/dev/storm/apache-storm-2.0.0-rc7/apache-storm-2.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-2.0.0-rc7/
>
> 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-1079
>
> Please vote on releasing this package as Apache Storm 2.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 2.0.0
> [ ]  0 No opinion
> [ ] -1 Do not release this package because...
>
> Thanks to everyone who contributed to this release.
>
> -Taylor