uranusjr commented on a change in pull request #15794:
URL: https://github.com/apache/airflow/pull/15794#discussion_r631015404
##########
File path: docs/apache-airflow-providers-apache-spark/connections/spark.rst
##########
@@ -27,7 +27,7 @@ The Apache Spark connection type enables connection to Apache
Spark.
Default Connection IDs
----------------------
-Spark Submit and Spark JDBC hooks and operators use ``spark_default`` by
default, Spark SQL hooks and operators point to ``spark_sql_default`` by
default, but don't use it.
+Spark Submit and Spark JDBC hooks and operators use ``spark_default`` by
default, Spark SQL hooks and operators point to ``spark_sql_default`` by
default, but its usage is optional.
Review comment:
Hmm yeah `SparkSubmitHook` seems to also follow this approach. I guess
that’s an expected approach then!
--
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.
For queries about this service, please contact Infrastructure at:
[email protected]
