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

[asfgit]

Github user asfgit closed the pull request at:

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


---

[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




---