adnanhemani commented on code in PR #4627:
URL: https://github.com/apache/polaris/pull/4627#discussion_r3383991112
##########
site/content/in-dev/unreleased/getting-started/setup-tools.md:
##########
@@ -37,7 +37,7 @@ This command supports bootstrapping new environments and
exporting existing conf
If you already have an Apache Polaris environment, you can export its current
state to a YAML file using the `export` subcommand:
```bash
-polaris setup export > polaris_bootstrap.yaml
+polaris setup export --client-id ${CLIENT_ID} --client-secret ${CLIENT_SECRET}
> polaris_bootstrap.yaml
Review Comment:
It is the intended way to use this script (or at the minimum, make it
explicit). We also don't give how to define a `polaris_bootstrap.yaml` file in
this page - so I currently read this page as more advisory than a step-by-step
guide. WDYT?
--
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]
