Fix coder parsing for any non-standard composite coder types#34606
Fix coder parsing for any non-standard composite coder types#34606lostluck merged 3 commits intoapache:masterfrom
Conversation
|
Assigning reviewers. If you would like to opt out of this review, comment R: @lostluck for label go. Available commands:
The PR bot will only process comments in the main thread (not review comments). |
|
We need to also un-skip the packed coders test in Python with this PR. Basically delete the overload test_pack_combiners here: https://github.com/apache/beam/blob/master/sdks/python/apache_beam/runners/portability/prism_runner_test.py#L220 |
…Composite are now mutually exclusive.
Done. |
|
Confirmed that test_pack_combiners runs and succeeds! The current failure seems like an unrelated flake, but we should re-run that to be sure, before merging in. |
fixes #32636
Inspired by #34348 (comment) and some offline discussions with @lostluck.