[VOTE] Release Apache Storm 1.2.2 (rc3)

2018-05-17 Thread 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:

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

2018-05-17 Thread 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; ---

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

2018-05-17 Thread 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; ---

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

2018-05-17 Thread 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

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

2018-05-17 Thread 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

2018-05-17 Thread 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

2018-05-17 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/2677 ---

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

2018-05-17 Thread 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

2018-05-17 Thread 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....

2018-05-17 Thread 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: