mchades commented on code in PR #9569:
URL: https://github.com/apache/gravitino/pull/9569#discussion_r2693218985


##########
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:
   You should describe what the recommended behavior is after deprecation.



##########
docs/manage-fileset-metadata-using-gravitino.md:
##########
@@ -921,3 +918,44 @@ curl -X POST -H "Accept: 
application/vnd.gravitino.v1+json" \
 }' 
http://localhost:8090/api/metalakes/test/catalogs/fileset_catalog/schemas/test_schema/filesets
 ```
 
+Alternatively, you can pass configurations for multiple clusters using the 
`fs.path.config.{cluster_name}.*` format when creating the fileset catalog.

Review Comment:
   Which has a higher priority: the priority of this configuration, or the 
priority of the same configuration on the fileset?



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