TheNeuralBit commented on pull request #15818: URL: https://github.com/apache/beam/pull/15818#issuecomment-953325521
I tested that this has the intended effect by running some Dataflow jobs that do a `sample()` at the beginning of a series of DataFrame operations, both with and without this change. Without the change:  Note that we are assigning partitions and shuffling at the beginning of the first stage, which is unnecessary. With the change:  The first stage is executed _before_ the first shuffle (also it looks like we need to do some escaping in the Python SDK? Including the file path in the name has lead to some artificial composites) -- 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]
