lordgamez commented on a change in pull request #1191:
URL: https://github.com/apache/nifi-minifi-cpp/pull/1191#discussion_r733457919



##########
File path: extensions/azure/processors/PutAzureBlobStorage.cpp
##########
@@ -242,10 +242,6 @@ void PutAzureBlobStorage::onTrigger(const 
std::shared_ptr<core::ProcessContext>
 
   std::optional<storage::UploadBlobResult> upload_result;
   {

Review comment:
       This block can be removed now and the declaration of upload_result can 
be moved to line 254.

##########
File path: extensions/azure/processors/PutAzureDataLakeStorage.cpp
##########
@@ -126,8 +126,6 @@ void PutAzureDataLakeStorage::onTrigger(const 
std::shared_ptr<core::ProcessConte
 
   storage::UploadDataLakeStorageResult result;
   {

Review comment:
       We can do the same here as well.




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


Reply via email to