LuciferYang opened a new pull request, #14407:
URL: https://github.com/apache/gravitino/pull/14407

   ### What changes were proposed in this pull request?
   
   `FileSystemCacheKey` also holds the values of the authentication settings in 
the merged catalog, schema and fileset configuration: `authentication.type`, 
`authentication.impersonation-enable`, `authentication.kerberos.principal`, 
`authentication.kerberos.keytab-uri`, `hadoop.security.authentication` (plain 
and with the `gravitino.bypass.` prefix) and `config.resources`. These are the 
per-fileset settings that decide how `HDFSFileSystemProxy` logs in.
   
   ### Why are the changes needed?
   
   Two filesets of one user could share a file system built for different 
authentication settings. Fileset configurations without these settings get the 
same key as before.
   
   Fix: #14405
   
   ### Does this PR introduce _any_ user-facing change?
   
   No.
   
   ### How was this patch tested?
   
   New `testFileSystemCacheSeparatesAuthenticationSettings`: for an absolute 
and a scheme-less path, configurations that differ in principal realm, 
authentication type or `gravitino.bypass.hadoop.security.authentication` get 
separate file systems, and configurations that differ only in other keys share 
one.
   


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