bamnet commented on pull request #12061: URL: https://github.com/apache/beam/pull/12061#issuecomment-653157515
I'm very new to Beam so don't have strong thoughts here, but two things jump out: 1) It's a breaking change. 2) LegacySQL appears to be the default for Python and Java. I don't know enough about the project to understand if there are too many users for 1 to be feasible (my gut sayz no) or if consistency with other language SDKs 2 is goal or non-goal. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
