@tweise The dataflow runner already use the field, of course we can make the 
breaking change and fix it in dataflow by tagging but to do a smooth 
transition, I hope to keep the URL field and remove in a subsequent PR.
Absolutely. Once its done,I will squash the PR appropriately. 

@mxm Absolutely. I will mark URL as deprecated and get rid of it ASAP.
Yes, I think I missed the GO code. I wonder why it did not break. Probably we 
need to regenerate the proto code as generated code is also checked in.


[ Full content available at: https://github.com/apache/beam/pull/6373 ]
This message was relayed via gitbox.apache.org for [email protected]

Reply via email to