yuqi1129 commented on PR #4248:
URL: https://github.com/apache/gravitino/pull/4248#issuecomment-2249824567

   > It's the best to have the abstraction like
   > 
   > ```
   > class UserContext {
   >      UserGroupInformation getUGI();
   >      doAs();
   > }
   > 
   > class SimpleUserContext implement UserContext {
   > }
   > 
   > class KerberosUserContext implement UserContext {
   > }
   > ```
   
   I'm okay with it, how to handle `impersonation`?  Add a property like 
`impersonation` to keep the value.
   
   Another point, by this meaning, is that we still need to use a map to 
calculate the real value of `impersonation` for a schema or fileset? The way 
you propose is a wapper that wapper `UGI` and `doAs` operation.


-- 
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]

Reply via email to