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:
[email protected]
