[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-548351399
 
 
   > Can we have this disable by default via a config flag. This is a debug facility and should remain so.
   
   Depends what is `debug facility` ;-) As I understood this posts the stack trace only in case of an unexpected exception. All of the expected exceptions are handled by OMException-s where the stack trace is not included.
   
   I think it's good to have the stack trace in these cases. Would be better to have it in a structured form, but as it's explained this is the easiest way. 
    
   Let me commit this, and we can continue the discussion if any configuration key is required (and if it is, we can add it in a follow-up jira)

----------------------------------------------------------------
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: hdfs-dev-unsubscribe@hadoop.apache.org
For additional commands, e-mail: hdfs-dev-help@hadoop.apache.org