This is an automated email from the ASF dual-hosted git repository.
yao pushed a commit to branch branch-4.0
in repository https://gitbox.apache.org/repos/asf/spark.git
The following commit(s) were added to refs/heads/branch-4.0 by this push:
new e918c10792f4 [SPARK-51861][SQL][UI] Remove duplicated/unnecessary info
of InMemoryRelation Plan Detail
e918c10792f4 is described below
commit e918c10792f4fcc37f2bc0923bbaa7a734ba4e0a
Author: Kent Yao <y...@apache.org>
AuthorDate: Tue Apr 22 13:31:46 2025 +0800
[SPARK-51861][SQL][UI] Remove duplicated/unnecessary info of
InMemoryRelation Plan Detail
### What changes were proposed in this pull request?
Remove duplicated info in InMemoryRelation Plan Detail
- serializer, a dev-only api
- cachedPlan, duplicated
- logicalPlan, not necessary for the physical detail
### Why are the changes needed?
These parameters are unnecessary and unformatted, which make the UI hard to
read.
### Does this PR introduce _any_ user-facing change?
UI changes
### How was this patch tested?
### Was this patch authored or co-authored using generative AI tooling?
no
Closes #50656 from yaooqinn/stringArgs.
Authored-by: Kent Yao <y...@apache.org>
Signed-off-by: Kent Yao <y...@apache.org>
(cherry picked from commit 7904dd3cd612721815d1cbc7d585efaac66d45a1)
Signed-off-by: Kent Yao <y...@apache.org>
---
.../org/apache/spark/sql/execution/columnar/InMemoryRelation.scala | 3 +++
1 file changed, 3 insertions(+)
diff --git
a/sql/core/src/main/scala/org/apache/spark/sql/execution/columnar/InMemoryRelation.scala
b/sql/core/src/main/scala/org/apache/spark/sql/execution/columnar/InMemoryRelation.scala
index 0f280d236203..bccc2799ba90 100644
---
a/sql/core/src/main/scala/org/apache/spark/sql/execution/columnar/InMemoryRelation.scala
+++
b/sql/core/src/main/scala/org/apache/spark/sql/execution/columnar/InMemoryRelation.scala
@@ -467,4 +467,7 @@ case class InMemoryRelation(
override def simpleString(maxFields: Int): String =
s"InMemoryRelation [${truncatedString(output, ", ", maxFields)}],
${cacheBuilder.storageLevel}"
+
+ override def stringArgs: Iterator[Any] =
+ Iterator(output, cacheBuilder.storageLevel, outputOrdering)
}
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@spark.apache.org
For additional commands, e-mail: commits-h...@spark.apache.org
