You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@nifi.apache.org by GitBox <gi...@apache.org> on 2020/02/23 20:22:43 UTC
[GitHub] [nifi] joewitt commented on issue #3984: NIFI-6970 add
DistributeRecord processor
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:
users@infra.apache.org
With regards,
Apache Git Services