adnanhemani commented on code in PR #1522:
URL: https://github.com/apache/polaris/pull/1522#discussion_r2074241794


##########
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:
   I'd just use `USER_CLIENT_ID` and `USER_CLIENT_ID` - and we should extend 
this change to Trino below as well. We're doing the same thing there.



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

Reply via email to