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(updated the Jira task type) 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.
   
   I think 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.
   
   If we are fine with it, I can update PR to improve Put/Delete and not change 
behavior for Fetch. 

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