diqiu50 commented on code in PR #9569:
URL: https://github.com/apache/gravitino/pull/9569#discussion_r2697574706
##########
catalogs/catalog-fileset/src/main/java/org/apache/gravitino/catalog/fileset/FilesetCatalogOperations.java:
##########
@@ -164,21 +175,19 @@ public class FilesetCatalogOperations extends
ManagedSchemaOperations
static class FileSystemCacheKey {
// When the path is a path without scheme such as 'file','hdfs', etc.,
then the scheme and
// authority are both null
+
+ // NOTE: The filesystem cache key contains the schema, authority and
current user.
+ // Changed the configuration of the same fileset. will not be effected by
the cached
+ // filesystem.
Review Comment:
fixed
##########
docs/fileset-catalog-with-adls.md:
##########
@@ -30,13 +30,12 @@ Apart from configurations mentioned in
[fileset-catalog-catalog-configuration](.
| Configuration item | Description
| Default value | Required | Since version |
|-------------------------------|------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------|-----------------|----------|------------------|
-| `filesystem-providers` | The file system providers to add. Set it to
`abs` if it's a Azure Blob Storage fileset, or a comma separated string that
contains `abs` like `oss,abs,s3` to support multiple kinds of fileset including
`abs`.
| (none) | Yes | 0.8.0-incubating |
-| `default-filesystem-provider` | The name default filesystem providers of
this Fileset catalog if users do not specify the scheme in the URI. Default
value is `builtin-local`, for Azure Blob Storage, if we set this value, we can
omit the prefix 'abfss://' in the location.
| `builtin-local` | No |
0.8.0-incubating |
+| `filesystem-providers` | (deprected) The file system providers to
add. Set it to `abs` if it's a Azure Blob Storage fileset, or a comma separated
string that contains `abs` like `oss,abs,s3` to support multiple kinds of
fileset including `abs`.
| (none) | Yes |
0.8.0-incubating |
Review Comment:
fixed
--
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]
