[GitHub] [spark] zsxwing commented on a change in pull request #25292: [SPARK-28556][SQL] QueryExecutionListener should also notify Error

[GitBox <gi...@apache.org>]

zsxwing commented on a change in pull request #25292: [SPARK-28556][SQL] QueryExecutionListener should also notify Error
URL: https://github.com/apache/spark/pull/25292#discussion_r308436939
 
 

 ##########
 File path: sql/core/src/main/scala/org/apache/spark/sql/util/QueryExecutionListener.scala
 ##########
 @@ -58,12 +58,12 @@ trait QueryExecutionListener {
    * @param funcName the name of the action that triggered this query.
    * @param qe the QueryExecution object that carries detail information like logical plan,
    *           physical plan, etc.
-   * @param exception the exception that failed this query.
+   * @param error the exception that failed this query.
 
 Review comment:
   Good catch

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org