FANNG1 commented on PR #5974:
URL: https://github.com/apache/gravitino/pull/5974#issuecomment-2577898948
My main concern is how to process Azure credentials in
`AzureFilesystemProviders`, the current implement seems hacky. How about add a
helper method to get credential related configurations in
`GravitinoVirtualFilesystem`? With this, we could remove all credential
process logic in `XXFilesystemProviders`.
```java
Map<String, String> credentialConfigs(String schema, Credential[]
credentials) {
}
```
--
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]
