[GitHub] [beam] ihji commented on pull request #22004: [BEAM-22003] Allow merging consecutive external transforms in Java SDK

[GitBox <gi...@apache.org>]

ihji commented on PR #22004:
URL: https://github.com/apache/beam/pull/22004#issuecomment-1181200727

   > I don't think we want the user to manually create an External transform that is a union of External transforms--the user might not even know which transforms are external and which ones are not. Instead, we should allow these transforms to be composed arbitrarily.
   > 
   > This may require creating a new type/coder in Java that represents the encoded form of the external type.
   
   I also thought the similar approach but wasn't sure if it's good idea to add the dummy (virtually do nothing but just placeholder) coder for Python objects to the Java SDK. So instead I implemented smaller, more limited scoped feature only allowing to manually combine consecutive external transforms.
   
   


-- 
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: github-unsubscribe@beam.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org