This is an automated email from the ASF dual-hosted git repository.
gengliang pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/spark.git
The following commit(s) were added to refs/heads/master by this push:
new b924e689942d [SPARK-48124][CORE] Disable structured logging for
Connect-Repl by default
b924e689942d is described below
commit b924e689942d735f165d31660d26efad057f4827
Author: panbingkun <panbing...@baidu.com>
AuthorDate: Sat May 4 22:47:24 2024 -0700
[SPARK-48124][CORE] Disable structured logging for Connect-Repl by default
### What changes were proposed in this pull request?
The pr is followup https://github.com/apache/spark/pull/46383, to `disable`
structured logging for` Connect-Repl` by default.
### Why are the changes needed?
Before:
<img width="1397" alt="image"
src="https://github.com/apache/spark/assets/15246973/10d93a09-f098-4653-9e95-571481dd03e9";>
After:
<img width="1406" alt="image"
src="https://github.com/apache/spark/assets/15246973/e3354359-d6bc-4b2c-801b-8a2c3697f78e";>
### Does this PR introduce _any_ user-facing change?
No.
### How was this patch tested?
Manually test.
### Was this patch authored or co-authored using generative AI tooling?
No.
Closes #46387 from panbingkun/SPARK-48124_FOLLOWUP.
Authored-by: panbingkun <panbing...@baidu.com>
Signed-off-by: Gengliang Wang <gengli...@apache.org>
---
.../main/scala/org/apache/spark/sql/application/ConnectRepl.scala | 5 +++++
1 file changed, 5 insertions(+)
diff --git
a/connector/connect/client/jvm/src/main/scala/org/apache/spark/sql/application/ConnectRepl.scala
b/connector/connect/client/jvm/src/main/scala/org/apache/spark/sql/application/ConnectRepl.scala
index 0360a4057886..9fd3ae4368f4 100644
---
a/connector/connect/client/jvm/src/main/scala/org/apache/spark/sql/application/ConnectRepl.scala
+++
b/connector/connect/client/jvm/src/main/scala/org/apache/spark/sql/application/ConnectRepl.scala
@@ -26,6 +26,7 @@ import ammonite.compiler.iface.CodeWrapper
import ammonite.util.{Bind, Imports, Name, Util}
import org.apache.spark.annotation.DeveloperApi
+import org.apache.spark.internal.Logging
import org.apache.spark.sql.SparkSession
import org.apache.spark.sql.connect.client.{SparkConnectClient,
SparkConnectClientParser}
@@ -55,6 +56,10 @@ object ConnectRepl {
inputStream: InputStream = System.in,
outputStream: OutputStream = System.out,
errorStream: OutputStream = System.err): Unit = {
+ // For interpreters, structured logging is disabled by default to avoid
generating mixed
+ // plain text and structured logs on the same console.
+ Logging.disableStructuredLogging()
+
// Build the client.
val client =
try {
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@spark.apache.org
For additional commands, e-mail: commits-h...@spark.apache.org
