yuqi1129 commented on PR #6059: URL: https://github.com/apache/gravitino/pull/6059#issuecomment-2586987047
> > > > IMO, credential vending is the critical part for the fileset use example for cloud storage, but now it lays in some inconspicuous places :( > > > > > > > > > Do you have any suggestions? The credential part is already with Header 2. do you mean I should move forward this part? > > > > > > Instead of placing it in a separate part at the last which seems optional and not important, place it in the catalog properties part like `### Configurations for S3 Hadoop Catalog`, for most examples use credential vending by default. > > * I don't believe putting them last means they are not important. > * The configuration items for Hadoop catalog in `hadoop-catalog.md` and for GVFS in `credential-vending.md`, Do I need make a copy here? Other points are - Credential vending may be quite an advanced feature and should be cautious in its usage. I'm afraid this will make primary users difficult to use the example. - I'm not very sure if we need to put it in the basic example just like I have put S3 example in the `hadoop-catalog.md` which is intentional for HDFS. If you have any further thoughts on it, please let me know your thoughts and ideas. -- 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]
