supratimdeka commented on issue #12: HDDS-2206. Separate handling for OMException and IOException in the Ozone Manager. Contributed by Supratim Deka URL: https://github.com/apache/hadoop-ozone/pull/12#issuecomment-543113365 2 modifications in latest commit: 1. Removed the configuration parameter. Marton had a review comment about additional complexity because of a config parameter. Also, making the configuration available in the validateAndUpdateCache(HDDS-2208) phase is quite complex. 2. Exception stack trace is propagated in the message field inside OMResponse. Throwing a ServiceException directly from the validateAndUpdateCache phase is not possible because of Ratis. It is very simple to dump the exception stack trace inside the message field. So we are always using the message field of OMResponse for exception stacktraces, to ensure uniformity between preExecute and validateAndUpdateCache.
---------------------------------------------------------------- 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 With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: hdfs-dev-unsubscr...@hadoop.apache.org For additional commands, e-mail: hdfs-dev-h...@hadoop.apache.org
