Abacn commented on code in PR #30587:
URL: https://github.com/apache/beam/pull/30587#discussion_r1523877743
##########
sdks/python/apache_beam/runners/portability/fn_api_runner/translations.py:
##########
@@ -538,6 +570,7 @@ def maybe_length_prefixed_and_safe_coder(self, coder_id):
spec=coder.spec, component_coder_ids=safe_component_ids))
return new_coder_id, safe_coder_id
else:
+ # A completely unkown coder. Wrap the entire thing in a length prefix.
Review Comment:
I see, the bytes encountered in XVR_Direct test was due to here
--
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]
