dongjoon-hyun commented on code in PR #50655:
URL: https://github.com/apache/spark/pull/50655#discussion_r2053232077
##########
sql/connect/server/src/main/scala/org/apache/spark/sql/connect/service/SparkConnectService.scala:
##########
@@ -363,7 +363,7 @@ object SparkConnectService extends Logging {
* Starts the GRPC Service.
*/
private def startGRPCService(): Unit = {
- val debugMode =
SparkEnv.get.conf.getBoolean("spark.connect.grpc.debug.enabled", true)
+ val debugMode =
SparkEnv.get.conf.getBoolean("spark.connect.grpc.debug.enabled", false)
Review Comment:
If Apache Spark still needs to depend on this flag somehow in testing
environment, we had better make it enabled only when `SPARK_TESTING=1`.
--
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: reviews-unsubscr...@spark.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
