flyrain commented on code in PR #3964: URL: https://github.com/apache/polaris/pull/3964#discussion_r2914687240
########## site/content/in-dev/unreleased/getting-started/using-polaris/_index.md: ########## @@ -179,7 +179,7 @@ docker attach $(docker ps -q --filter name=spark-sql) Once the Spark session starts, we can create a namespace and table within the catalog: ```sql -USE quickstart_catalog; +USE polaris; Review Comment: is this place setting the spark config? https://github.com/apache/polaris/blob/1e7bb1a6b077df4f7494925ed0a89ed7f09c3c59/getting-started/quickstart/docker-compose.yml#L269 If yes, we can add this line to there, and remove the line `USE polaris;` here. ``` --conf spark.sql.defaultCatalog=polaris ``` We may add this to line 151 anyways, so that both work. WDYT? ``` --conf spark.sql.defaultCatalog=quickstart_catalog ``` -- 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]
