You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@iceberg.apache.org by ao...@apache.org on 2020/10/29 18:11:25 UTC

[iceberg] branch master updated: Spark: Enable testInsertOverwrite, passing with Spark 3.0.1 (#1679)

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 8886bad  Spark: Enable testInsertOverwrite, passing with Spark 3.0.1 (#1679)
8886bad is described below

commit 8886bad6831181c42b4c5a44ff9091e2ce7d1503
Author: Ryan Blue <bl...@apache.org>
AuthorDate: Thu Oct 29 11:11:10 2020 -0700

    Spark: Enable testInsertOverwrite, passing with Spark 3.0.1 (#1679)
---
 .../test/java/org/apache/iceberg/spark/sql/TestPartitionedWrites.java  | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/spark3/src/test/java/org/apache/iceberg/spark/sql/TestPartitionedWrites.java b/spark3/src/test/java/org/apache/iceberg/spark/sql/TestPartitionedWrites.java
index 80584b1..9223797 100644
--- a/spark3/src/test/java/org/apache/iceberg/spark/sql/TestPartitionedWrites.java
+++ b/spark3/src/test/java/org/apache/iceberg/spark/sql/TestPartitionedWrites.java
@@ -31,7 +31,6 @@ import org.apache.spark.sql.functions;
 import org.junit.After;
 import org.junit.Assert;
 import org.junit.Before;
-import org.junit.Ignore;
 import org.junit.Test;
 
 public class TestPartitionedWrites extends SparkCatalogTestBase {
@@ -69,7 +68,7 @@ public class TestPartitionedWrites extends SparkCatalogTestBase {
     assertEquals("Row data should match expected", expected, sql("SELECT * FROM %s ORDER BY id", tableName));
   }
 
-  @Ignore // broken because of SPARK-32168, which should be fixed in 3.0.1
+  @Test
   public void testInsertOverwrite() {
     Assert.assertEquals("Should have 3 rows", 3L, scalarSql("SELECT count(*) FROM %s", tableName));