sushilkm edited a comment on issue #3906: NIFI-6911 Removed default value for 
Blob property for AzureBlobStorage processors
URL: https://github.com/apache/nifi/pull/3906#issuecomment-559601878
 
 
   I would agree about it is not exactly a bug and it makes sense for 
List/Fetch combination, however for PutAzureBlobStorage/DeleteAzureBlobStorage 
it still will make more sense to not have that default, because they are 
generally not used in conjunction with ListAzureBlobStorage.
   
   The difference between overriding the default value is, if you do not 
override and go with default for these two processors, and a user goes with 
filling up the empty fields, processor gives impression that everything is ok 
and only way user finds problem is after starting that default value is invalid.
   
   To me a default value should be something which makes some sense does not 
needs an override almost always, however in this case 
(Put/Delete)AzureBlobStorage almost always need to override default with 
DeleteAzureBlobStorage having an exception in case of being used with 
ListDeleteAzureBlobStorage.

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


With regards,
Apache Git Services

Reply via email to