amaliujia edited a comment on pull request #13898: URL: https://github.com/apache/beam/pull/13898#issuecomment-780880231
My suggestion is: 1. if this PR's goal is to verify whether BeamCalRel is safe to run ZetaSQL operators, can you create a separate rule that allows ZetaSQL operators run in BeamCalRel(and reject those wrong cases)? 2. If this PR's goal is to help improve Java UDF, please hold this PR now. Kyle and I are discussing the next steps to improve current UDF implementation. This PR is a great reference to give us directions on where to check the implementation. We will need time to digest this PR and consider whether Calc splitting should go first. We will definitely consider this PR during our improving process. ---------------------------------------------------------------- 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]
