[GitHub] [incubator-kyuubi] jiaoqingbo commented on a diff in pull request #3641: [KYUUBI #3640] Change the default event logger description of hive and trino to json instead of spark

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

jiaoqingbo commented on code in PR #3641:
URL: https://github.com/apache/incubator-kyuubi/pull/3641#discussion_r996674666


##########
kyuubi-common/src/main/scala/org/apache/kyuubi/config/KyuubiConf.scala:
##########
@@ -2109,7 +2109,7 @@ object KyuubiConf {
   val ENGINE_TRINO_EVENT_LOGGERS: ConfigEntry[Seq[String]] =
     buildConf("kyuubi.engine.trino.event.loggers")
       .doc("A comma separated list of engine history loggers, where engine/session/operation etc" +
-        " events go. We use spark logger by default.<ul>" +
+        " events go. We use json logger by default.<ul>" +

Review Comment:
   nice 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.

To unsubscribe, e-mail: notifications-unsubscribe@kyuubi.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@kyuubi.apache.org
For additional commands, e-mail: notifications-help@kyuubi.apache.org