markap14 commented on code in PR #5944:
URL: https://github.com/apache/nifi/pull/5944#discussion_r876045387
##########
nifi-nar-bundles/nifi-azure-bundle/nifi-azure-processors/src/main/java/org/apache/nifi/processors/azure/storage/FetchAzureBlobStorage.java:
##########
@@ -85,6 +79,22 @@ public class FetchAzureBlobStorage extends
AbstractAzureBlobProcessor {
.required(false)
.build();
+ private static final List<PropertyDescriptor> properties =
Collections.unmodifiableList(Arrays.asList(
+ AzureStorageUtils.CONTAINER_WITH_DEFAULT_VALUE,
+ BLOB,
+ AzureStorageUtils.STORAGE_CREDENTIALS_SERVICE,
+ AzureStorageUtils.ACCOUNT_NAME,
+ AzureStorageUtils.ACCOUNT_KEY,
+ AzureStorageUtils.PROP_SAS_TOKEN,
+ AzureStorageUtils.ENDPOINT_SUFFIX,
+ RANGE_START,
+ RANGE_LENGTH,
+ AzureBlobClientSideEncryptionUtils.CSE_KEY_TYPE,
+ AzureBlobClientSideEncryptionUtils.CSE_KEY_ID,
+ AzureBlobClientSideEncryptionUtils.CSE_SYMMETRIC_KEY_HEX,
Review Comment:
They are related to the blob. But they are far less important. Meaning that
they will be configured far less often. The most important (read: the most
commonly configured) properties should be first.
--
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]
