bharathv commented on a change in pull request #934: HBASE-23568: Improve Threading of Replication URL: https://github.com/apache/hbase/pull/934#discussion_r376666733
########## File path: hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/ReplicationSourceWALReader.java ########## @@ -409,17 +413,7 @@ private boolean acquireBufferQuota(long size) { return totalBufferUsed.addAndGet(size) >= totalBufferQuota; } - /** - * @return whether the reader thread is running - */ - public boolean isReaderRunning() { - return isReaderRunning && !isInterrupted(); Review comment: Shouldn't it also be looking for an interrupt? (like in a thread pool shutdown). ---------------------------------------------------------------- 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