dlmarion commented on code in PR #4761:
URL: https://github.com/apache/accumulo/pull/4761#discussion_r1691562537


##########
core/src/main/java/org/apache/accumulo/core/iterators/SynchronizedServerFilter.java:
##########
@@ -48,13 +49,19 @@ public 
SynchronizedServerFilter(SortedKeyValueIterator<Key,Value> source) {
 
   @Override
   public synchronized void next() throws IOException {
+    if (Thread.interrupted()) {

Review Comment:
   I'm not sure what the penalty is for calling this for every K/V. I'm also 
wondering if the message should be logged as well in case the RuntimeException 
is ignored in a higher level iterator.



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

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to