[GitHub] [pulsar] zjyingchang commented on a change in pull request #10163: Add Thread.currentThread.interrupt() after catching InterruptedException and document it in the javadoc

[GitBox]

zjyingchang commented on a change in pull request #10163:
URL: https://github.com/apache/pulsar/pull/10163#discussion_r610986444



##########
File path: 
pulsar-client/src/main/java/org/apache/pulsar/client/impl/ConsumerImpl.java
##########
@@ -453,6 +455,7 @@ public UnAckedMessageTracker getUnAckedMessageTracker() {
         try {
             return internalBatchReceiveAsync().get();
         } catch (InterruptedException | ExecutionException e) {
+            Thread.currentThread().interrupt();

Review comment:
       @eolivelli Please take a look again!  I add a Util class to handle 
InterruptedException and use it in XXXConsumeImpl.receive() and 
XXXConsumeImpl.receiveBatch() , or I need only add this Util method in 
ConsumeBase.  




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