amaechler opened a new pull request, #16561:
URL: https://github.com/apache/druid/pull/16561

   ### Description
   
   This PR marks two additional properties in the Azure deep storage config as 
required (`@NotNull`) (also see [the 
docs](https://druid.apache.org/docs/latest/development/extensions-core/azure/)):
   
   - `druid.azure.account`
   - `druid.azure.container`
   
   Both these properties must be set. I also cleaned up the 
`AzureAccountConfig` class a bit (no more equality overrides, use JSON 
serialization instead in tests to verify equality), group getters / setters 
together, and add some Javadocs to the `getBlobStorageEndpoint` helper function.
   
   <hr>
   
   This PR has:
   
   - [x] been self-reviewed.
   - [x] been tested in a test Druid cluster.
   


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

Reply via email to