apurtell commented on a change in pull request #2574:
URL: https://github.com/apache/hbase/pull/2574#discussion_r510317299
##########
File path:
hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java
##########
@@ -4550,6 +4619,10 @@ private void doMiniBatchMutate(BatchOperation<?>
batchOp) throws IOException {
/** Keep track of the locks we hold so we can release them in finally
clause */
List<RowLock> acquiredRowLocks =
Lists.newArrayListWithCapacity(batchOp.size());
try {
+ // Check for thread interrupt status in case we have been signaled from
+ // #interruptRegionOperation.
+ checkInterrupt();
Review comment:
Yes, but on the other hand we are basically checking interrupt status
manually, as one has to do in Java, so can check and exit early at only what we
think are safe points, which is nice.
----------------------------------------------------------------
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]
