eric-maynard commented on code in PR #1522:
URL: https://github.com/apache/polaris/pull/1522#discussion_r2072827406
##########
site/content/in-dev/unreleased/getting-started/using-polaris.md:
##########
@@ -172,9 +172,8 @@ Finally, note that we include the `hadoop-aws` package
here. If your table is us
Replace the credentials used in the Docker container using the following code:
```shell
-USER_CLIENT_ID="XXXX"
-USER_CLIENT_SECRET="YYYY"
-sed -i
"s/^\(.*spark\.sql\.catalog\.quickstart_catalog\.credential=\).*/\1${USER_CLIENT_ID}:${USER_CLIENT_SECRET}\",/"
getting-started/eclipselink/docker-compose.yml
+export USER_CLIENT_ID="XXXX"
Review Comment:
.env is a good idea, I wish I would have thought of that!
@owenowenisme you're right that this is a one-time thing... but that's also
why modifying the `getting-started/eclipselink/docker-compose.yml` is suspect.
Maybe we could write to a new file and `source` it instead of using the
normal .env? Or we could just append to the end of the existing .env.
--
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: issues-unsubscr...@polaris.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
