[GitHub] [nifi] IlyaKovalev commented on issue #3984: NIFI-6970 add DistributeRecord processor

[GitBox <gi...@apache.org>]

IlyaKovalev commented on issue #3984: NIFI-6970 add DistributeRecord processor
URL: https://github.com/apache/nifi/pull/3984#issuecomment-575571931
 
 
   1 Fixed, also remove ORIGINAL relationship (it's excessively i think)
   2 Yes, i really thought about it. In case merge logic of DistributeRecord with DistributeLoad we should be add reader, writer, keys, hash_function, strategy fields but logic for understanding how processor will process input is growing i suggest it's too ambigious.
   So look at this like:
   DistributeLoad - works on flowfile level without processing content i.e distribute flowfiles.
   DistributeRecord - works on content level i.e distribute content
   (So maybe we need rename DistributeLoad to DistributeFlowFile just because DistributeLoad is 
   very general definition)
   I think it's much easier for understanding processor logic(how it would works) and much simplest for user to appreciate init process.

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


With regards,
Apache Git Services