[GitHub] [hadoop-ozone] elek commented on issue #47: HDDS-2208. Propagate System Exceptions from OM transaction apply phase. Contributed by Supratim Deka

[GitBox <gi...@apache.org>]

elek commented on issue #47: HDDS-2208. Propagate System Exceptions from OM transaction apply phase. Contributed by Supratim Deka
URL: https://github.com/apache/hadoop-ozone/pull/47#issuecomment-549305769
 
 
   I discussed it with @anuengineer and understood that he has more concerns regarding the propagation of stack traces to the client, as it couldn't be used easily with any non-java client in the future.
   
   I also found this comment: https://issues.apache.org/jira/browse/HDDS-2206
   
   > 
   > Reverted this based on offline conversation with Anu Engineer.
   > 
   > Anu has requested we add a config key to control this behavior.
   > 
   
   But it's hard to understand the concerns based on this comment line. 
   
   I propose to create short design-document about the vision of error handling to make it easier to follow in all the patches.
   
   We can adjust the behavior in a follow-up patch. (For example: instead of propagating the stack trace we can improve the insight tool to help to find the real error without propagation).   

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


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: ozone-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: ozone-issues-help@hadoop.apache.org