ahmedabu98 commented on issue #34797:
URL: https://github.com/apache/beam/issues/34797#issuecomment-2845057371

   When Dataflow service catches up, this new parameter should work fine.. but 
I think at the same time it would break older SDK versions that don't have the 
new parameter. Either way, there will be an imbalance according to 
`assignableTo()`.
   
   I wonder if this logic can be improved to ignore nullable fields? cc 
@chamikaramj 


-- 
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-unsubscr...@beam.apache.org

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

Reply via email to