yaooqinn commented on code in PR #44575:
URL: https://github.com/apache/spark/pull/44575#discussion_r1441185899
##########
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveUtils.scala:
##########
@@ -201,6 +201,16 @@ private[spark] object HiveUtils extends Logging {
.booleanConf
.createWithDefault(true)
+ val HIVE_THRIFT_SERVER_EXIT_ON_ERROR =
+ buildConf("spark.sql.hive.thriftServer.exitOnThriftCLIServiceError")
Review Comment:
Thank you for pointing this out, @dongjoon-hyun. I completely agree with you
that creating a new configuration for a test fix would be excessive. However,
in this case, we can address both the DevelopAPI and test side issue as a
typical use case simultaneously, This way, the solution would not be limited to
just fixing the test.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
