egalpin commented on PR #24887:
URL: https://github.com/apache/beam/pull/24887#issuecomment-1526870609
@kennknowles thanks for the bump. I have not had much bandwidth to
meaningfully test this change. Something that concerns me and requires
validation is whether or not the coder can be successfully set/changed from
default so as to allow backward compatibility for pipeline authors. It appears
that using .setCoder was not entirely sufficient based on prior exploration[1],
but I have not yet revisited or gotten into a good repro-test feedback loop.
I have no issue handing this off if it’s high priority and someone else is
keen to take up the work.
[1] https://github.com/apache/beam/issues/21162#issuecomment-1371259467
--
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]
