bowenliang123 commented on PR #3419: URL: https://github.com/apache/incubator-kyuubi/pull/3419#issuecomment-1237861254
> > Back in #3343 , these ut were intended to verify someone who has select privilege on source tables have the create temp view rights. Plus Authz is checking output tables after input tables. With select from table, we could verify input table privilege checks pass and then output tables. So maybe we should keep the original ut and add ut for `user_w/o_permission`. They maybe vary in cases. > > I think we just need to check to see if we skipped the temporary view check Alright, I am neutral here. -- 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]
