You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@storm.apache.org by agresch <gi...@git.apache.org> on 2018/05/17 19:34:15 UTC
[GitHub] storm pull request #2684: STORM-3079 add getMessage() support for thrift exc...
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 <ag...@...>
Date: 2018-05-17T19:31:20Z
STORM-3079 add getMessage() support for thrift exceptions
----
---
[GitHub] storm pull request #2684: STORM-3079 add getMessage() support for thrift exc...
Posted by revans2 <gi...@git.apache.org>.
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...
Posted by asfgit <gi...@git.apache.org>.
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...
Posted by agresch <gi...@git.apache.org>.
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
---