You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@seatunnel.apache.org by GitBox <gi...@apache.org> on 2022/10/20 13:05:30 UTC

[GitHub] [incubator-seatunnel] EricJoy2048 commented on pull request #3045: [Improve][Core][Exceptiom Management] Add Exception Management API & Unified Exception in API Module

EricJoy2048 commented on PR #3045:
URL: https://github.com/apache/incubator-seatunnel/pull/3045#issuecomment-1285507665

   Hi, all. What is our conclusion? I agree with @TyrantLucifer , The exception handling of `http-server <-> app-client` is also the result of continuous optimization and iteration, and I think their design is very good for user experience. ErrorCode can simple help user to understand what happened in the system.  On the other hand, ErrorCode can reduce randomly defined exceptions. 
   
   Especially when we need to collect error data in the future, ErrorCode can be stored in the error data together with the data. When we display the cause of the error in the web page, the ErrorCode can obtain the details of the exception, which is very helpful for the analysis of error data.


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@seatunnel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org