Github user ijokarumawak commented on the issue:

    https://github.com/apache/nifi/pull/2384
  
    Hi @martin-mucha Thanks for your first contribution! The PR is closed 
without being reviewed and I can't find other PRs created for NIFI-4745. Do you 
have a PR ready for being reviewed somewhere?
    
    About your concern to add another argument to the completeFlowFile method 
which has already lots of arguments, I think that should be fine, since it's 
just a private method and the method is a good place to add new attribute 
values.
    
    BTW, ValidateRecord can validate multiple Records within a single incoming 
FlowFile. Is it going to be sufficient to write validation result to a FlowFile 
attribute? Isn't it be more useful if we can write validation failure detail to 
the outgoing Records, so that each record can have its failure detail?



---

Reply via email to