suneet-s commented on code in PR #15770:
URL: https://github.com/apache/druid/pull/15770#discussion_r1470093759
##########
extensions-core/azure-extensions/src/main/java/org/apache/druid/storage/azure/AzureStorage.java:
##########
@@ -188,6 +188,12 @@ public void batchDeleteFiles(String containerName,
Iterable<String> paths, Integ
);
}
+ public void batchDeleteFiles(String containerName, Iterable<String> paths)
+ throws BlobBatchStorageException
+ {
+ batchDeleteFiles(containerName, paths, null);
+ }
Review Comment:
nit: No need for this function we can call the existing one with null
directly. It would be good to add javadocs on the function above that
`maxAttempts` is null able and users should call that with null when they want
to use the system configured max retries.
```suggestion
```
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
