juliuszsompolski commented on a change in pull request #25960:
[SPARK-29283][SQL] Error message is hidden when query from JDBC, especially
enabled adaptive execution
URL: https://github.com/apache/spark/pull/25960#discussion_r330549260
##########
File path:
sql/hive-thriftserver/src/main/scala/org/apache/spark/sql/hive/thriftserver/SparkGetCatalogsOperation.scala
##########
@@ -68,11 +69,18 @@ private[hive] class SparkGetCatalogsOperation(
}
setState(OperationState.FINISHED)
} catch {
- case e: HiveSQLException =>
+ case e: Throwable =>
Review comment:
Hm. I guess we may want to catch a Throwable.
E.g. InterruptedExpression is not catched by NonFatal, and we want to inform
the HiveThriftServer2.listener about the error after an interrupt - this
definitely can happen in SparkExecuteStatementOperation that is async and can
be cancelled. After a ThreadDeath of OutOfMemoryError I think we also want to
inform the HiveThriftServer2.listener to not get the query hanging in the UI,
as I think the server would continue to go on (I think it won't bring the whole
JVM down?).
----------------------------------------------------------------
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]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
