AngersZhuuuu commented on PR #5699: URL: https://github.com/apache/kyuubi/pull/5699#issuecomment-1820307068
> The case in PR desc looks reasonable to me. Anything belonging to the table shall have proper privileges. If you use * to retrieve them, then authorize them. Otherwise, * shall not point to the meta columns instead of skipping them in authz. Oh...I make a mistake, just checked with the hudi dev that in our prod, `SELECT * FROM HUDI` will ignore the HUDI meta column, but for real case, it will show hudi's meta columns too. Since we write unit test with `collect()` not `show()`, didn't find this. <img width="1170" alt="截屏2023-11-21 下午2 18 38" src="https://github.com/apache/kyuubi/assets/46485123/51239ee6-dd5f-45ee-9841-89b13af0f57f";> -- 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]
