You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@iceberg.apache.org by GitBox <gi...@apache.org> on 2022/11/21 14:27:22 UTC

[GitHub] [iceberg] Fokko opened a new pull request, #6239: Docs: Select the right Spark catalog

Fokko opened a new pull request, #6239:
URL: https://github.com/apache/iceberg/pull/6239

   I copy pasted the example, but still had to select the 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: issues-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org


[GitHub] [iceberg] rdblue commented on a diff in pull request #6239: Docs: Select the right Spark catalog

Posted by GitBox <gi...@apache.org>.
rdblue commented on code in PR #6239:
URL: https://github.com/apache/iceberg/pull/6239#discussion_r1051691004


##########
docs/aws.md:
##########
@@ -68,6 +68,7 @@ done
 
 # start Spark SQL client shell
 spark-sql --packages $DEPENDENCIES \
+    --conf spark.sql.defaultCatalog=my_catalog \

Review Comment:
   Agreed. This would be a good follow up.



-- 
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-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org


[GitHub] [iceberg] rdblue merged pull request #6239: Docs: Select the right Spark catalog

Posted by GitBox <gi...@apache.org>.
rdblue merged PR #6239:
URL: https://github.com/apache/iceberg/pull/6239


-- 
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-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org


[GitHub] [iceberg] ajantha-bhat commented on a diff in pull request #6239: Docs: Select the right Spark catalog

Posted by GitBox <gi...@apache.org>.
ajantha-bhat commented on code in PR #6239:
URL: https://github.com/apache/iceberg/pull/6239#discussion_r1028370385


##########
docs/aws.md:
##########
@@ -68,6 +68,7 @@ done
 
 # start Spark SQL client shell
 spark-sql --packages $DEPENDENCIES \
+    --conf spark.sql.defaultCatalog=my_catalog \

Review Comment:
   In the same file there are multiple `spark-sql` commands, we need to add to all of these and also check in the other files? 



-- 
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-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org


[GitHub] [iceberg] ajantha-bhat commented on a diff in pull request #6239: Docs: Select the right Spark catalog

Posted by GitBox <gi...@apache.org>.
ajantha-bhat commented on code in PR #6239:
URL: https://github.com/apache/iceberg/pull/6239#discussion_r1028370385


##########
docs/aws.md:
##########
@@ -68,6 +68,7 @@ done
 
 # start Spark SQL client shell
 spark-sql --packages $DEPENDENCIES \
+    --conf spark.sql.defaultCatalog=my_catalog \

Review Comment:
   In the same file there are multiple `spark-sql` commands, we need to add to all of these and also check in the other files? :) 



-- 
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-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org


[GitHub] [iceberg] nastra commented on a diff in pull request #6239: Docs: Select the right Spark catalog

Posted by GitBox <gi...@apache.org>.
nastra commented on code in PR #6239:
URL: https://github.com/apache/iceberg/pull/6239#discussion_r1030464928


##########
docs/aws.md:
##########
@@ -68,6 +68,7 @@ done
 
 # start Spark SQL client shell
 spark-sql --packages $DEPENDENCIES \
+    --conf spark.sql.defaultCatalog=my_catalog \

Review Comment:
   +1 to updating the other places with the same setting



-- 
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-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org