[GitHub] storm pull request #2692: STORM-3083 Upgrade HikariCP version to 2.4.7

2018-05-24 Thread asfgit 
Github user asfgit closed the pull request at:

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


---

[GitHub] storm pull request #2693: STORM-3084 fix Nimbus NPE

2018-05-24 Thread agresch 
GitHub user agresch opened a pull request:

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

STORM-3084 fix Nimbus NPE



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

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

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

https://github.com/apache/storm/pull/2693.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 #2693


commit fe388df9b3e9649fbdf9f2e57206ee655212bcc6
Author: Aaron Gresch 
Date:   2018-05-24T21:39:07Z

STORM-3084 fix Nimbus NPE




---

[GitHub] storm pull request #2688: STORM-3061: Remove unused core dependencies

2018-05-24 Thread asfgit 
Github user asfgit closed the pull request at:

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


---

[GitHub] storm issue #2691: STORM-3061: Update version of hbase

2018-05-24 Thread revans2 
Github user revans2 commented on the issue:

https://github.com/apache/storm/pull/2691
  
Yes that was the plan.  there is a lot that depends on storm-autocreds and 
I would like to understand it all better before I try to clean it up.


---

[GitHub] storm issue #2691: STORM-3061: Update version of hbase

2018-05-24 Thread arunmahadevan 
Github user arunmahadevan commented on the issue:

https://github.com/apache/storm/pull/2691
  
Actually I was wrong. storm-hdfs, storm-hbase, storm-hive seem to depend on 
storm-autocreds. I guess we could pull out the required classes into some 
common package as part of the follow up JIRA.


---

[GitHub] storm issue #2640: [STORM-3036] Add isRemoteBlobExists RPC interface for dec...

2018-05-24 Thread danny0405 
Github user danny0405 commented on the issue:

https://github.com/apache/storm/pull/2640
  
@revans2 
Hi, Bobby, kindly reminder.


---

[GitHub] storm issue #2691: STORM-3061: Update version of hbase

2018-05-24 Thread arunmahadevan 
Github user arunmahadevan commented on the issue:

https://github.com/apache/storm/pull/2691
  
Sure we can revisit this in a follow up JIRA.

We may not have to split the autocreds since none of the other components 
depends on it. The hbase-server dependency if included is just going to end up 
under external/storm-autocreds and not going to be included in the class path 
by default. We could also check with the Hbase team to pull out the TokenUtils 
into hbase-client package.


---

[GitHub] storm pull request #780: STORM-1086. Make FailedMsgRetryManager configurable...

2018-05-24 Thread kleinsch 
Github user kleinsch closed the pull request at:

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


---

[GitHub] storm issue #2691: STORM-3061: Update version of hbase

2018-05-24 Thread revans2 
Github user revans2 commented on the issue:

https://github.com/apache/storm/pull/2691
  
@arunmahadevan I am happy to try and split up autocreds to make that 
happen, but it is a much larger job than what is currently for this.  If you 
are fine with waiting I would rather file a follow on JIRA to upgrade to 2.0.0 
and split up autocreds instead of blocking this.


---