[VOTE] Release Apache Storm 1.2.2 (rc3)

[P. Taylor Goetz]

This is a call to vote on releasing Apache Storm 1.2.2 (rc3)

Full list of changes in this release:

https://dist.apache.org/repos/dist/dev/storm/apache-storm-1.2.2-rc3/RELEASE_NOTES.html

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

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

The source archive being voted upon can be found here:

https://dist.apache.org/repos/dist/dev/storm/apache-storm-1.2.2-rc3/apache-storm-1.2.2-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.2-rc3/

The release artifacts are signed with the following key:

https://git-wip-us.apache.org/repos/asf?p=storm.git;a=blob_plain;f=KEYS;hb=22b832708295fa2c15c4f3c70ac0d2bc6fded4bd

The Nexus staging repository for this release is:

https://repository.apache.org/content/repositories/orgapachestorm-1068

Please vote on releasing this package as Apache Storm 1.2.2.

When voting, please list the actions taken to verify the release.

This vote will be open for 72 hours or until at least 3 PMC members vote +1.

[ ] +1 Release this package as Apache Storm 1.2.2
[ ]  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

[GitHub] storm pull request #2684: STORM-3079 add getMessage() support for thrift exc...

[agresch]

Github user agresch commented on a diff in the pull request:

https://github.com/apache/storm/pull/2684#discussion_r189079432
  
--- Diff: 
storm-client/src/jvm/org/apache/storm/utils/WrappedAuthorizationException.java 
---
@@ -0,0 +1,17 @@
+package org.apache.storm.utils;
--- End diff --

Fixed


---

[GitHub] storm pull request #2684: STORM-3079 add getMessage() support for thrift exc...

[revans2]

Github user revans2 commented on a diff in the pull request:

https://github.com/apache/storm/pull/2684#discussion_r189076711
  
--- Diff: 
storm-client/src/jvm/org/apache/storm/utils/WrappedAuthorizationException.java 
---
@@ -0,0 +1,17 @@
+package org.apache.storm.utils;
--- End diff --

Missing the license header.


---

[GitHub] storm pull request #2684: STORM-3079 add getMessage() support for thrift exc...

[agresch]

GitHub user agresch opened a pull request:

https://github.com/apache/storm/pull/2684

STORM-3079 add getMessage() support for thrift exceptions



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/agresch/storm agresch_thriftexception

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/storm/pull/2684.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #2684


commit 48451e5d6855635fa5237140f0c7b7c8a79ebd67
Author: Aaron Gresch 
Date:   2018-05-17T19:31:20Z

STORM-3079 add getMessage() support for thrift exceptions




---

[GitHub] storm issue #2669: [STORM-3055] remove conext connection cache

[pczb]

Github user pczb commented on the issue:

https://github.com/apache/storm/pull/2669
  
@srdo @revans2 thanks for your review, i just squash all commit in one


---

[GitHub] storm issue #2669: [STORM-3055] remove conext connection cache

[revans2]

Github user revans2 commented on the issue:

https://github.com/apache/storm/pull/2669
  
Once you make the corresponding changes on #2661 I would be happy to merge 
them both in.


---

[GitHub] storm pull request #2677: STORM-3075 fix NPE

[asfgit]

Github user asfgit closed the pull request at:

https://github.com/apache/storm/pull/2677


---

[GitHub] storm issue #2677: STORM-3075 fix NPE

[agresch]

Github user agresch commented on the issue:

https://github.com/apache/storm/pull/2677
  
@Ethanlm - done.  Added a spelling fix.


---

[GitHub] storm issue #2677: STORM-3075 fix NPE

[Ethanlm]

Github user Ethanlm commented on the issue:

https://github.com/apache/storm/pull/2677
  
@agresch  you can squash the commits and then I can merge it in.


---

[GitHub] storm pull request #2683: STORM-3077 Upgrade Disruptor version to 3.3.11 (1....

[HeartSaVioR]

GitHub user HeartSaVioR opened a pull request:

https://github.com/apache/storm/pull/2683

STORM-3077 Upgrade Disruptor version to 3.3.11 (1.x)

No need to port forward to 2.x since we no longer use Disruptor in 2.x.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/HeartSaVioR/storm STORM-3077-1.x

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/storm/pull/2683.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #2683


commit 4f7840eb03e6519683d710ff5ac6262dd04a546f
Author: Jungtaek Lim 
Date:   2018-05-17T11:24:46Z

STORM-3077 Upgrade Disruptor version to 3.3.11




---