lostluck commented on PR #30368: URL: https://github.com/apache/beam/pull/30368#issuecomment-1971640098
I'd say while partitions are inherently numerically based, that's largely an implementation detail to serve the user goal of "turn 1 PCollection<T> into N PCollection<T>s, known at Pipeline Construction Time". Personally I'd say that the "standard" beam one is just a "PartitionByIndex", vs a general "PartitionBy" transform that this appears to be. I have no objection to devlist bikeshedding though. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
