Conversation
|
|
||
| ```sql | ||
| USE quickstart_catalog; | ||
| USE polaris; |
There was a problem hiding this comment.
I think quickstart_catalog is consistent with the value in the spark config above, see line 151.
Another option is to have this config in the spark config around line 151, so that we don't need the use xxx command.
--conf spark.sql.defaultCatalog=quickstart_catalog
There was a problem hiding this comment.
@flyrain Gotcha! Thanks for the feedback! I'll take a closer look on your suggestion over the weekend.
There was a problem hiding this comment.
this is another place where #2963 broke the quickstart. The docker-compose.yml file states "polaris" as the catalog name but the Spark block above states "quickstart_catalog".
There was a problem hiding this 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
|
Maybe I missed something, but from the current getting start guide, it will still point users to |
|
Yes, we need to fix the URL to the docker-compose YAML file 👍 |
@MonkeyCanCode - Yes, we ran into this issue yesterday with the paths. @adnanhemani said he has to take a closer look as to whether updating I can update the docs over the weekend if there's a consensus of what the correct path forward is. |
|
We should point all "getting-started/" references into "site/content/guides". |
I think this is the only one where we are using a absolute URL. In other places such as the aws/gcp/azure ones, it is still pointing to |
Updating documentation. The command has been updated to from
quickstart_catalogtopolaris.Checklist
CHANGELOG.md(if needed)site/content/in-dev/unreleased(if needed)