(iceberg) branch main updated: Core, Spark 3.4: Remove redundant output in tests (#10348)

[aokolnychyi]

This is an automated email from the ASF dual-hosted git repository.

aokolnychyi pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/iceberg.git


The following commit(s) were added to refs/heads/main by this push:
     new 2886ef4bf6 Core, Spark 3.4: Remove redundant output in tests (#10348)
2886ef4bf6 is described below

commit 2886ef4bf6cf575f9780a5bfd351a4f4d51cce4b
Author: Anton Okolnychyi <aokolnyc...@apple.com>
AuthorDate: Fri May 17 10:20:57 2024 -0700

    Core, Spark 3.4: Remove redundant output in tests (#10348)
---
 .../java/org/apache/iceberg/metrics/TestCommitMetricsResultParser.java  | 1 -
 .../java/org/apache/iceberg/metrics/TestScanMetricsResultParser.java    | 1 -
 .../extensions/TestSystemFunctionPushDownInRowLevelOperations.java      | 2 --
 3 files changed, 4 deletions(-)

diff --git 
a/core/src/test/java/org/apache/iceberg/metrics/TestCommitMetricsResultParser.java
 
b/core/src/test/java/org/apache/iceberg/metrics/TestCommitMetricsResultParser.java
index 7598555c6b..5aa2660143 100644
--- 
a/core/src/test/java/org/apache/iceberg/metrics/TestCommitMetricsResultParser.java
+++ 
b/core/src/test/java/org/apache/iceberg/metrics/TestCommitMetricsResultParser.java
@@ -231,7 +231,6 @@ public class TestCommitMetricsResultParser {
     String expectedJson = "{ }";
 
     String json = CommitMetricsResultParser.toJson(commitMetricsResult, true);
-    System.out.println(json);
     assertThat(json).isEqualTo(expectedJson);
     assertThat(CommitMetricsResultParser.fromJson(json))
         .isEqualTo(ImmutableCommitMetricsResult.builder().build());
diff --git 
a/core/src/test/java/org/apache/iceberg/metrics/TestScanMetricsResultParser.java
 
b/core/src/test/java/org/apache/iceberg/metrics/TestScanMetricsResultParser.java
index 47b67cd961..2daa2b4592 100644
--- 
a/core/src/test/java/org/apache/iceberg/metrics/TestScanMetricsResultParser.java
+++ 
b/core/src/test/java/org/apache/iceberg/metrics/TestScanMetricsResultParser.java
@@ -324,7 +324,6 @@ public class TestScanMetricsResultParser {
     String expectedJson = "{ }";
 
     String json = ScanMetricsResultParser.toJson(scanMetricsResult, true);
-    System.out.println(json);
     Assertions.assertThat(json).isEqualTo(expectedJson);
     Assertions.assertThat(ScanMetricsResultParser.fromJson(json))
         .isEqualTo(ImmutableScanMetricsResult.builder().build());
diff --git 
a/spark/v3.4/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestSystemFunctionPushDownInRowLevelOperations.java
 
b/spark/v3.4/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestSystemFunctionPushDownInRowLevelOperations.java
index db4d10645b..49119d319c 100644
--- 
a/spark/v3.4/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestSystemFunctionPushDownInRowLevelOperations.java
+++ 
b/spark/v3.4/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestSystemFunctionPushDownInRowLevelOperations.java
@@ -317,8 +317,6 @@ public class TestSystemFunctionPushDownInRowLevelOperations 
extends SparkExtensi
   private List<Expression> executeAndCollectFunctionCalls(String query, 
Object... args) {
     CommandResultExec command = (CommandResultExec) executeAndKeepPlan(query, 
args);
     V2TableWriteExec write = (V2TableWriteExec) command.commandPhysicalPlan();
-    System.out.println("!!! WRITE PLAN !!!");
-    System.out.println(write.toString());
     return SparkPlanUtil.collectExprs(
         write.query(),
         expr -> expr instanceof StaticInvoke || expr instanceof 
ApplyFunctionExpression);