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

amoghj 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 fb657b413e Spark: Simplify SparkSchemaUtil#schemaForTable (#10137)
fb657b413e is described below

commit fb657b413e2bb7f6c5e2c78465173df0426d3527
Author: Amogh Jahagirdar <am...@tabular.io>
AuthorDate: Sun Apr 14 07:44:28 2024 -0600

    Spark: Simplify SparkSchemaUtil#schemaForTable (#10137)
---
 .../spark/src/main/java/org/apache/iceberg/spark/SparkSchemaUtil.java | 4 +---
 .../spark/src/main/java/org/apache/iceberg/spark/SparkSchemaUtil.java | 4 +---
 .../spark/src/main/java/org/apache/iceberg/spark/SparkSchemaUtil.java | 4 +---
 3 files changed, 3 insertions(+), 9 deletions(-)

diff --git 
a/spark/v3.3/spark/src/main/java/org/apache/iceberg/spark/SparkSchemaUtil.java 
b/spark/v3.3/spark/src/main/java/org/apache/iceberg/spark/SparkSchemaUtil.java
index 6075aba7ac..b1cc0ac620 100644
--- 
a/spark/v3.3/spark/src/main/java/org/apache/iceberg/spark/SparkSchemaUtil.java
+++ 
b/spark/v3.3/spark/src/main/java/org/apache/iceberg/spark/SparkSchemaUtil.java
@@ -59,9 +59,7 @@ public class SparkSchemaUtil {
    * @return a Schema for the table, if found
    */
   public static Schema schemaForTable(SparkSession spark, String name) {
-    StructType sparkType = spark.table(name).schema();
-    Type converted = SparkTypeVisitor.visit(sparkType, new 
SparkTypeToType(sparkType));
-    return new Schema(converted.asNestedType().asStructType().fields());
+    return convert(spark.table(name).schema());
   }
 
   /**
diff --git 
a/spark/v3.4/spark/src/main/java/org/apache/iceberg/spark/SparkSchemaUtil.java 
b/spark/v3.4/spark/src/main/java/org/apache/iceberg/spark/SparkSchemaUtil.java
index 21a1d92a11..d0f77bcdd9 100644
--- 
a/spark/v3.4/spark/src/main/java/org/apache/iceberg/spark/SparkSchemaUtil.java
+++ 
b/spark/v3.4/spark/src/main/java/org/apache/iceberg/spark/SparkSchemaUtil.java
@@ -59,9 +59,7 @@ public class SparkSchemaUtil {
    * @return a Schema for the table, if found
    */
   public static Schema schemaForTable(SparkSession spark, String name) {
-    StructType sparkType = spark.table(name).schema();
-    Type converted = SparkTypeVisitor.visit(sparkType, new 
SparkTypeToType(sparkType));
-    return new Schema(converted.asNestedType().asStructType().fields());
+    return convert(spark.table(name).schema());
   }
 
   /**
diff --git 
a/spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/SparkSchemaUtil.java 
b/spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/SparkSchemaUtil.java
index 21a1d92a11..d0f77bcdd9 100644
--- 
a/spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/SparkSchemaUtil.java
+++ 
b/spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/SparkSchemaUtil.java
@@ -59,9 +59,7 @@ public class SparkSchemaUtil {
    * @return a Schema for the table, if found
    */
   public static Schema schemaForTable(SparkSession spark, String name) {
-    StructType sparkType = spark.table(name).schema();
-    Type converted = SparkTypeVisitor.visit(sparkType, new 
SparkTypeToType(sparkType));
-    return new Schema(converted.asNestedType().asStructType().fields());
+    return convert(spark.table(name).schema());
   }
 
   /**

Reply via email to