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:
[email protected]
