arpadboda commented on a change in pull request #826:
URL: https://github.com/apache/nifi-minifi-cpp/pull/826#discussion_r447269150



##########
File path: libminifi/src/FlowFileRecord.cpp
##########
@@ -366,7 +366,7 @@ bool FlowFileRecord::DeSerialize(const uint8_t *buffer, 
const int bufferSize) {
     return false;
   }
 
-  if (nullptr == claim_) {

Review comment:
       Yep, the goal would be to *always* have claim, even if the content is 
empty.
   The reason behind is that we don't have to deal with failing streams and 
claims when handling flowfiles. A good example of this is PutFile or 
PublishKafka, where handling both types of "empty" flowfiles is a pain. 




----------------------------------------------------------------
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:
us...@infra.apache.org


Reply via email to