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

ravipesala 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 e485e14  [SDV] Fix SDV testcase failures
e485e14 is described below

commit e485e14a8a3f680190aedaa540755a432ea4d7c3
Author: shivamasn <shivamas...@gmail.com>
AuthorDate: Thu Jan 17 18:12:23 2019 +0530

    [SDV] Fix SDV testcase failures
    
    problem: Dictionary Include test cases were failing because the plan was 
incorrect.
    
    solution: Updated the plan in the QueryTest.
    
    This closes #3081
---
 .../src/test/scala/org/apache/spark/sql/common/util/QueryTest.scala   | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git 
a/integration/spark-common-cluster-test/src/test/scala/org/apache/spark/sql/common/util/QueryTest.scala
 
b/integration/spark-common-cluster-test/src/test/scala/org/apache/spark/sql/common/util/QueryTest.scala
index 3ee75ad..39beae1 100644
--- 
a/integration/spark-common-cluster-test/src/test/scala/org/apache/spark/sql/common/util/QueryTest.scala
+++ 
b/integration/spark-common-cluster-test/src/test/scala/org/apache/spark/sql/common/util/QueryTest.scala
@@ -44,8 +44,6 @@ class QueryTest extends PlanTest with Suite {
 
   // Add Locale setting
   Locale.setDefault(Locale.US)
-  CarbonProperties.getInstance()
-    .addProperty(CarbonCommonConstants.VALIDATE_DIRECT_QUERY_ON_DATAMAP, 
"false")
 
   /**
    * Runs the plan and makes sure the answer contains all of the keywords, or 
the
@@ -139,7 +137,7 @@ class QueryTest extends PlanTest with Suite {
 
   val sqlContext: SQLContext = TestQueryExecutor.INSTANCE.sqlContext
 
-  sqlContext.sparkSession.experimental.extraOptimizations = Seq(new 
CarbonFileIndexReplaceRule)
+  sqlContext.sparkSession.experimental.extraOptimizations :+ Seq(new 
CarbonFileIndexReplaceRule)
 
   val resourcesPath = TestQueryExecutor.resourcesPath
 

Reply via email to