[GitHub] [nifi] turcsanyip commented on pull request #3906: NIFI-6911 Removed the default value for blob property for PutAzureBlobStorage

[GitBox <gi...@apache.org>]

turcsanyip commented on pull request #3906:
URL: https://github.com/apache/nifi/pull/3906#issuecomment-621488579


   My opinion still is that removing the default value is not a good idea. It is quite common that Put file/blob object processors have the core attribute `${filename}` as their default value. It exists on all FlowFiles so it cannot happen that it has no value (unlike `${azure.blobname}` which is not a standard attribute).
   
   The other Azure Blob processors could also be modified to use this `${filename}` (while preserving the legacy `${azure.blobname}` as well for List/Fetch in order to maintain backward compatibility). In this way, the Azure Blob processors would be standardized to the `${filename}` convention. Furthermore, the property descriptor overriding (BLOB vs BLOB_NAME) would not be needed because all of them would use the same default value.


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org