rmatharu commented on a change in pull request #1511:
URL: https://github.com/apache/samza/pull/1511#discussion_r677864641
##########
File path:
samza-azure/src/main/java/org/apache/samza/system/azureblob/avro/AzureBlobOutputStream.java
##########
@@ -335,6 +335,11 @@ public void run() {
// StageBlock generates exception on Failure.
stageBlock(blockIdEncoded, outputStream, blockSize);
break;
+ } catch (InterruptedException e) {
+ String msg = "Upload block for blob: " +
blobAsyncClient.getBlobUrl().toString()
+ + " failed for blockid: " + blockId + " due to
InterruptedException ";
+ LOG.error(msg, e);
+ throw new AzureException("InterruptedException encountered during
block upload. Will not retry.", e);
Review comment:
Is there a reason why on a thread being interrupted should it not be
retried?
--
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]
