virajjasani commented on a change in pull request #2172:
URL: https://github.com/apache/hbase/pull/2172#discussion_r463167959
##########
File path:
hbase-server/src/main/java/org/apache/hadoop/hbase/util/RegionMover.java
##########
@@ -587,8 +483,12 @@ private void waitMoveTasksToFinish(ExecutorService
moveRegionsPool,
LOG.error("Interrupted while waiting for Thread to Complete " +
e.getMessage(), e);
throw e;
} catch (ExecutionException e) {
- LOG.error("Got Exception From Thread While moving region " +
e.getMessage(), e);
- throw e;
+ if (e.getCause() instanceof UnknownRegionException) {
+ LOG.info("Ignore unknown region, it might have been split/merged.");
Review comment:
Sure, let me revert it back to debug and Exception message is the only
way to actually print region id here. Future contains only `Boolean` value
otherwise.
----------------------------------------------------------------
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:
[email protected]
