[GitHub] [zeppelin] zjffdu opened a new pull request #4031: [ZEPPELIN-5215]. Use TApplicationException instead of TException to propagate exception to zeppelin server

[GitBox]

zjffdu opened a new pull request #4031:
URL: https://github.com/apache/zeppelin/pull/4031


   
   ### What is this PR for?
   
   Currently we use TException in RemoteInterpreterServer which make zeppelin 
server unknown what kind of error happens in interpreter process side. Using 
TApplicationException can propagate the exception to zeppelin server side. 
   See 
https://livebook.manning.com/book/programmers-guide-to-apache-thrift/chapter-9/87
   
   ### What type of PR is it?
   [Improvement ]
   
   ### Todos
   * [ ] - Task
   
   ### What is the Jira issue?
   * https://issues.apache.org/jira/browse/ZEPPELIN-5215
   
   ### How should this be tested?
   * Manually tested
   
   ### Screenshots (if appropriate)
   
   Before 
   
![image](https://user-images.githubusercontent.com/164491/105442764-4c09c500-5ca5-11eb-8592-4954c3fa1f38.png)
   
   
   After
   
   
![image](https://user-images.githubusercontent.com/164491/105442437-cc7bf600-5ca4-11eb-914d-2be569358d31.png)
   
   ### Questions:
   * Does the licenses files need update? No
   * Is there breaking changes for older versions? No
   * Does this needs documentation? No
   


----------------------------------------------------------------
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:
us...@infra.apache.org