Bipin Prasad created STORM-3702:
-----------------------------------
Summary: Change thrift exception classes to contain getMessage()
method
Key: STORM-3702
URL: https://issues.apache.org/jira/browse/STORM-3702
Project: Apache Storm
Issue Type: Improvement
Components: storm-client
Affects Versions: 2.1.0
Reporter: Bipin Prasad
Assignee: Bipin Prasad
Thrift classes are currently being generated without camel casing for method
names. And the exception classes have a member variable called "msg". This
generates, get_msg() and set_msg() method names. Since this class extends java
Exception class, when an TException is thrown, the getMessage() method returns
null and cannot be relied upon.
To get around this problem with getMessage() method, storm code has Wrapper
classes, eg WrappedNotAliveException with a getMessage() method that return
wraps get_msg() method in base class.
Proposed Change:
* Change the TException classes to rename member variable and generate camel
cased class so that getMessage() is generated
* Limit the change to specific TException classes only to avoid large change
* Deprecate Wrapped classes
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
