markap14 commented on code in PR #5944:
URL: https://github.com/apache/nifi/pull/5944#discussion_r876025670
##########
nifi-nar-bundles/nifi-azure-bundle/nifi-azure-processors/src/main/java/org/apache/nifi/processors/azure/storage/PutAzureBlobStorage_v12.java:
##########
@@ -92,10 +97,10 @@ public class PutAzureBlobStorage_v12 extends
AbstractAzureBlobProcessor_v12 {
.build();
private static final List<PropertyDescriptor> PROPERTIES =
Collections.unmodifiableList(Arrays.asList(
- STORAGE_CREDENTIALS_SERVICE,
- AzureStorageUtils.CONTAINER,
- CREATE_CONTAINER,
- BLOB_NAME
+ AzureStorageUtils.CONTAINER,
+ BLOB_NAME,
+ CREATE_CONTAINER,
+ STORAGE_CREDENTIALS_SERVICE
Review Comment:
In reviewing this now after a while, I would actually argue that it makes
more sense to push the CREATE_CONTAINER lower in the list. When I decide I want
to push data to Azure Blob Storage, the first thing I want to configure is the
container to push it in and the name of the object. Then I want to configure
authentication. Whether or not the container should be auto-created is of
secondary importance.
--
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]
