saintstack opened a new pull request #1216: HBASE-23904 Procedure updating meta 
and Master shutdown are incompati…
URL: https://github.com/apache/hbase/pull/1216
 
 
   …ble: CODE-BUG
   
   Restore behavior from before HBASE-21789 (hbase-2.2.0) where we convert
   all exceptions to IOEs, even RuntimeExceptions. Actual fix is this change 
(in case
   obscured by doc and lambda simplification):
   
        } catch (Throwable e) {
   -      Throwables.propagateIfPossible(e, IOException.class);
   +      // Throw if an IOE else wrap in an IOE EVEN IF IT IS a 
RuntimeException (e.g.
   +      // a RejectedExecutionException because the hosting exception is 
shutting down.
   +      // This is old behavior worth reexamining. Procedures doing merge or 
split
   +      // currently don't handle RuntimeExceptions coming up out of meta 
table edits.
   +      // Would have to work on this at least. See HBASE-23904.
   +      Throwables.throwIfInstanceOf(e, IOException.class);

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

Reply via email to