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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to