georgew5656 commented on code in PR #16558: URL: https://github.com/apache/druid/pull/16558#discussion_r1628428060
########## extensions-core/azure-extensions/src/main/java/org/apache/druid/storage/azure/AzureStorage.java: ########## @@ -56,255 +54,391 @@ */ public class AzureStorage Review Comment: is it possible for a cluster using azure as deep storage to first make a read-only call to azure (therefore not creating the container) and for the function to throw a exception instead of creating the container and returning a empty list or something? like e.g if i start a new cluster in azure with no container what's the first api call that gets made and what happens if its a read-only call? i think i would be more concerned with error handling, if we return a good exception that says "the container doesn't exist" or something like this, then the operator could just create the container at that point. -- 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]
