bowenliang123 commented on PR #4168: URL: https://github.com/apache/kyuubi/pull/4168#issuecomment-1631028857
I will merge this PR first. And I would also like to see these changes in follow-ups: 1. separate ut cases for different usage of functions, rather than putting them all in one case 2. check the calling of unresolved/non-existed functions and existing functions 3. resolve the problem with 3.4 before implementation of checking function privileges in the plugin -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
