adamdebreceni commented on a change in pull request #807:
URL: https://github.com/apache/nifi-minifi-cpp/pull/807#discussion_r454831606



##########
File path: libminifi/src/core/ProcessSession.cpp
##########
@@ -211,15 +206,6 @@ std::shared_ptr<core::FlowFile> 
ProcessSession::clone(const std::shared_ptr<core
 
 void ProcessSession::remove(const std::shared_ptr<core::FlowFile> &flow) {
   flow->setDeleted(true);
-  if (flow->getResourceClaim() != nullptr) {
-    flow->getResourceClaim()->decreaseFlowFileRecordOwnedCount();
-    logger_->log_debug("Auto terminated %s %" PRIu64 " %s", 
flow->getResourceClaim()->getContentFullPath(), 
flow->getResourceClaim()->getFlowFileRecordOwnedCount(), flow->getUUIDStr());
-  } else {
-    logger_->log_debug("Flow does not contain content. no resource claim to 
decrement.");

Review comment:
       added comment where the delete actually happens in 
`ProcessSession::commit`




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


Reply via email to