Ethanlm commented on pull request #3337: URL: https://github.com/apache/storm/pull/3337#issuecomment-696758659
> @bipinprasad As discussed, I feel all the thrift API methods thould wrap these exception under `TException`. So we can consistently ensure message/cause is passed down to the client side. Currently sending just message is not sufficient/not follow typical exception criteria of providing cause. I would like to learn more about this approach. The current way of passing an exact exception class (like NotAliveException) seems fine with me ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected]
