justinmclean commented on PR #5058: URL: https://github.com/apache/gravitino/pull/5058#issuecomment-2426018345
From your feedback @shaofengshi most of this work has been in this PR, other PRs or is planned in future phases. I don't think anything else needs to be done in this Pr. 1. This has already been done, and a simplified CLI, as @jerryshao asked for, implemented, i.e. --command, --metalake, --schema etc, have been removed. 2. This has also already been done in the second PR, including support for setting it as an environment variable or a configuration file. 3. It is recommended that the user uses the full name; all examples and uses in the documentation use the full name. 4. The URL could be specified in an environment variable or the config file as part of the second PR or as a separate PR after that has been merged. Authentication is not part of this phase but is in phase 10, but it can be implemented as you suggest. -- 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]
