boittega commented on a change in pull request #15794:
URL: https://github.com/apache/airflow/pull/15794#discussion_r631003270
##########
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:
On both Spark connectors, you have the option to send all the parameters
and not using the connection as far as I know. Correct me if I am wrong.
##########
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:
On both Spark hooks, you have the option to send all the parameters and
not using the connection as far as I know. Correct me if I am wrong.
--
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]
