[carbondata] branch master updated: [CARBONDATA-4250] Ignoring presto random test cases

[in...@apache.org]

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 0337c32  [CARBONDATA-4250] Ignoring presto random test cases
0337c32 is described below

commit 0337c3268cb49a9a6d9283d25259350d7d6319a1
Author: Mahesh Raju Somalaraju <ma...@huawei.com>
AuthorDate: Mon Jul 19 19:38:46 2021 +0530

    [CARBONDATA-4250] Ignoring presto random test cases
    
    Why is this PR needed?
    Presto test cases failing randomly and taking more time in CI verification for other PRs.
    
    What changes were proposed in this PR?
    Currently presto random test cases will be ignored and will be fixed with other JIRA raised.
    1. JIRA [CARBONDATA-4250] raised for ignoring presto test cases currently as this random
       failures causing PR CI failures.
    2. JIRA [CARBONDATA-4249] raised for fixing presto random tests in concurrent scenario.
       We can get more details on this JIRA about issue reproduce and problem snippet.
    3. [CARBONDATA-4254] raised to fix Test alter add for structs enabling local dictionary
       and CarbonIndexFileMergeTestCaseWithSI.Verify command of index merge
    
    This closes #4176
---
 .../testsuite/mergeindex/CarbonIndexFileMergeTestCaseWithSI.scala     | 2 +-
 .../carbondata/presto/integrationtest/PrestoTestUsingSparkStore.scala | 3 ++-
 .../spark/testsuite/dataload/SparkStoreCreatorForPresto.scala         | 3 ++-
 .../spark/testsuite/alterTable/TestAlterTableAddColumns.scala         | 4 ++--
 4 files changed, 7 insertions(+), 5 deletions(-)

diff --git a/index/secondary-index/src/test/scala/org/apache/carbondata/spark/testsuite/mergeindex/CarbonIndexFileMergeTestCaseWithSI.scala b/index/secondary-index/src/test/scala/org/apache/carbondata/spark/testsuite/mergeindex/CarbonIndexFileMergeTestCaseWithSI.scala
index d2025fb..6963827 100644
--- a/index/secondary-index/src/test/scala/org/apache/carbondata/spark/testsuite/mergeindex/CarbonIndexFileMergeTestCaseWithSI.scala
+++ b/index/secondary-index/src/test/scala/org/apache/carbondata/spark/testsuite/mergeindex/CarbonIndexFileMergeTestCaseWithSI.scala
@@ -98,7 +98,7 @@ class CarbonIndexFileMergeTestCaseWithSI
       sql("""Select count(*) from indexmerge"""))
   }
 
-  test("Verify command of index merge") {
+  ignore("Verify command of index merge") {
     CarbonProperties.getInstance()
       .addProperty(CarbonCommonConstants.CARBON_MERGE_INDEX_IN_SEGMENT, "false")
     sql("DROP TABLE IF EXISTS nonindexmerge")
diff --git a/integration/presto/src/test/scala/org/apache/carbondata/presto/integrationtest/PrestoTestUsingSparkStore.scala b/integration/presto/src/test/scala/org/apache/carbondata/presto/integrationtest/PrestoTestUsingSparkStore.scala
index 67f2617..f699bcb 100644
--- a/integration/presto/src/test/scala/org/apache/carbondata/presto/integrationtest/PrestoTestUsingSparkStore.scala
+++ b/integration/presto/src/test/scala/org/apache/carbondata/presto/integrationtest/PrestoTestUsingSparkStore.scala
@@ -21,7 +21,7 @@ import java.io.{File}
 import java.util
 
 import org.apache.commons.io.FileUtils
-import org.scalatest.{BeforeAndAfterAll, BeforeAndAfterEach, FunSuiteLike}
+import org.scalatest.{BeforeAndAfterAll, BeforeAndAfterEach, FunSuiteLike, Ignore}
 
 import org.apache.carbondata.common.logging.LogServiceFactory
 import org.apache.carbondata.core.constants.CarbonCommonConstants
@@ -29,6 +29,7 @@ import org.apache.carbondata.core.datastore.impl.FileFactory
 import org.apache.carbondata.core.util.{CarbonProperties, CarbonUtil}
 import org.apache.carbondata.presto.server.{PrestoServer, PrestoTestUtil}
 
+@Ignore
 class PrestoTestUsingSparkStore
   extends FunSuiteLike with BeforeAndAfterAll with BeforeAndAfterEach {
 
diff --git a/integration/spark/src/test/scala/org/apache/carbondata/integration/spark/testsuite/dataload/SparkStoreCreatorForPresto.scala b/integration/spark/src/test/scala/org/apache/carbondata/integration/spark/testsuite/dataload/SparkStoreCreatorForPresto.scala
index 56a9c48..dc63475 100644
--- a/integration/spark/src/test/scala/org/apache/carbondata/integration/spark/testsuite/dataload/SparkStoreCreatorForPresto.scala
+++ b/integration/spark/src/test/scala/org/apache/carbondata/integration/spark/testsuite/dataload/SparkStoreCreatorForPresto.scala
@@ -25,7 +25,7 @@ import scala.util.Random
 import org.apache.commons.io.FileUtils
 import org.apache.spark.sql.CarbonEnv
 import org.apache.spark.sql.test.util.QueryTest
-import org.scalatest.BeforeAndAfterAll
+import org.scalatest.{BeforeAndAfterAll, Ignore}
 
 import org.apache.carbondata.core.constants.CarbonCommonConstants
 import org.apache.carbondata.core.datastore.impl.FileFactory
@@ -33,6 +33,7 @@ import org.apache.carbondata.core.util.{CarbonProperties, CarbonUtil}
 import org.apache.carbondata.core.util.path.CarbonTablePath
 import org.apache.carbondata.sdk.file.{CarbonSchemaReader, CarbonWriterBuilder}
 
+@Ignore
 class SparkStoreCreatorForPresto extends QueryTest with BeforeAndAfterAll{
 
   private val timestampFormat = CarbonProperties.getInstance()
diff --git a/integration/spark/src/test/scala/org/apache/carbondata/spark/testsuite/alterTable/TestAlterTableAddColumns.scala b/integration/spark/src/test/scala/org/apache/carbondata/spark/testsuite/alterTable/TestAlterTableAddColumns.scala
index 9a512f6..8f3497b 100644
--- a/integration/spark/src/test/scala/org/apache/carbondata/spark/testsuite/alterTable/TestAlterTableAddColumns.scala
+++ b/integration/spark/src/test/scala/org/apache/carbondata/spark/testsuite/alterTable/TestAlterTableAddColumns.scala
@@ -336,7 +336,7 @@ class TestAlterTableAddColumns extends QueryTest with BeforeAndAfterAll {
     assert(c(0)(2) == null)
   }
 
-  test("Test alter add for structs enabling local dictionary") {
+  ignore("Test alter add for structs enabling local dictionary") {
     createTableForComplexTypes("LOCAL_DICTIONARY_INCLUDE", "STRUCT")
     // For the previous segments the default value for newly added struct column is null
     insertIntoTableForStructType
@@ -346,7 +346,7 @@ class TestAlterTableAddColumns extends QueryTest with BeforeAndAfterAll {
     sql("DROP TABLE IF EXISTS alter_struct")
   }
 
-  test("Test alter add for structs, disabling local dictionary") {
+  ignore("Test alter add for structs, disabling local dictionary") {
     createTableForComplexTypes("LOCAL_DICTIONARY_EXCLUDE", "STRUCT")
     // For the previous segments the default value for newly added struct column is null
     insertIntoTableForStructType