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