joewitt commented on issue #3984: NIFI-6970 add DistributeRecord processor URL: https://github.com/apache/nifi/pull/3984#issuecomment-590111770 Hello. This is an interesting PR/good idea. I wonder though if this should just be a partitioning strategy in PartitionRecord. If we keep it separate like this we might want to go with a more specific name than DistributeRecord as we might want different distribution options and this one seems to be pretty specific to weighted distribution using one or more hashing functions. Maybe then the name should be 'WeightedRecordDistribution' In any case how about squashing the commits and rebasing to master and pushing the forced PR. This will let the new CI processes run against the PR.
---------------------------------------------------------------- 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] With regards, Apache Git Services
