beam icon indicating copy to clipboard operation
beam copied to clipboard

Fix Maybereshuffle

Open xliee opened this issue 1 year ago • 1 comments

This pull request includes changes to the expand method in the sdks/python/apache_beam/transforms/core.py file to simplify the code and remove an unnecessary inner class. The most important changes include the removal of the MaybeReshuffle inner class and the restructuring of the expand method.

Code simplification and restructuring:

  • sdks/python/apache_beam/transforms/core.py: Removed the MaybeReshuffle inner class and restructured the expand method to directly use pcol with Reshuffle if necessary.
  • sdks/python/apache_beam/transforms/core.py: Adjusted the return statement in the expand method to simplify the pipeline construction.

GitHub Actions Tests Status (on master branch)

Build python source distribution and wheels Python tests Java tests Go tests

See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.

xliee avatar Oct 15 '24 14:10 xliee

Checks are failing. Will not request review until checks are succeeding. If you'd like to override that behavior, comment assign set of reviewers

github-actions[bot] avatar Oct 15 '24 15:10 github-actions[bot]

This pull request has been marked as stale due to 60 days of inactivity. It will be closed in 1 week if no further activity occurs. If you think that’s incorrect or this pull request requires a review, please simply write any comment. If closed, you can revive the PR at any time and @mention a reviewer or discuss it on the [email protected] list. Thank you for your contributions.

github-actions[bot] avatar Dec 15 '24 12:12 github-actions[bot]

This pull request has been closed due to lack of activity. If you think that is incorrect, or the pull request requires review, you can revive the PR at any time.

github-actions[bot] avatar Dec 23 '24 12:12 github-actions[bot]