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


##########
site/content/in-dev/unreleased/getting-started/using-polaris.md:
##########
@@ -21,7 +21,12 @@ Title: Using Polaris
 type: docs
 weight: 400
 ---
-
+## Setup
+Define your `CLIENT_ID` & `CLIENT_SECRET` and put them in .env file for later 
use.

Review Comment:
   > Wait sorry, I just realized that this is already detailed in the 
[quickstart](https://polaris.apache.org/in-dev/unreleased/getting-started/quickstart/)
 page which links into here:
   > 
   > ```
   > When Polaris is run using the ./gradlew run command, the root principal 
credentials are root and secret for the CLIENT_ID and CLIENT_SECRET, 
respectively. You can also set these credentials as environment variables for 
use with the Polaris CLI:
   > 
   > export CLIENT_ID=root
   > export CLIENT_SECRET=secret
   > ```
   > 
   > So we now have redundant but inconsistent instructions when going from the 
quickstart into `using-polaris`
   
   I don't think this is conflicting.
   Using Polaris is another page.
   



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