Repository: spark
Updated Branches:
  refs/heads/master 8193a266b -> 14464cadb


[SPARK-14038][SQL] enable native view by default

## What changes were proposed in this pull request?

As we have completed the `SQLBuilder`, we can safely turn on native view by 
default.

## How was this patch tested?

existing tests.

Author: Wenchen Fan <[email protected]>

Closes #11872 from cloud-fan/native-view.


Project: http://git-wip-us.apache.org/repos/asf/spark/repo
Commit: http://git-wip-us.apache.org/repos/asf/spark/commit/14464cad
Tree: http://git-wip-us.apache.org/repos/asf/spark/tree/14464cad
Diff: http://git-wip-us.apache.org/repos/asf/spark/diff/14464cad

Branch: refs/heads/master
Commit: 14464cadb9477be8b7f4c891ea990535ab6638ec
Parents: 8193a26
Author: Wenchen Fan <[email protected]>
Authored: Tue Mar 22 00:07:57 2016 -0700
Committer: Reynold Xin <[email protected]>
Committed: Tue Mar 22 00:07:57 2016 -0700

----------------------------------------------------------------------
 .../main/scala/org/apache/spark/sql/internal/SQLConf.scala  | 2 +-
 .../spark/sql/hive/execution/HiveCompatibilitySuite.scala   | 9 ++++++---
 2 files changed, 7 insertions(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/spark/blob/14464cad/sql/core/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala
----------------------------------------------------------------------
diff --git 
a/sql/core/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala 
b/sql/core/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala
index 61058ea..70d1a8b 100644
--- a/sql/core/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala
+++ b/sql/core/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala
@@ -370,7 +370,7 @@ object SQLConf {
           "unmatching partitions can be eliminated earlier.")
 
   val NATIVE_VIEW = booleanConf("spark.sql.nativeView",
-    defaultValue = Some(false),
+    defaultValue = Some(true),
     doc = "When true, CREATE VIEW will be handled by Spark SQL instead of Hive 
native commands.  " +
           "Note that this function is experimental and should ony be used when 
you are using " +
           "non-hive-compatible tables written by Spark SQL.  The SQL string 
used to create " +

http://git-wip-us.apache.org/repos/asf/spark/blob/14464cad/sql/hive/compatibility/src/test/scala/org/apache/spark/sql/hive/execution/HiveCompatibilitySuite.scala
----------------------------------------------------------------------
diff --git 
a/sql/hive/compatibility/src/test/scala/org/apache/spark/sql/hive/execution/HiveCompatibilitySuite.scala
 
b/sql/hive/compatibility/src/test/scala/org/apache/spark/sql/hive/execution/HiveCompatibilitySuite.scala
index 0dc2a95..05f59f1 100644
--- 
a/sql/hive/compatibility/src/test/scala/org/apache/spark/sql/hive/execution/HiveCompatibilitySuite.scala
+++ 
b/sql/hive/compatibility/src/test/scala/org/apache/spark/sql/hive/execution/HiveCompatibilitySuite.scala
@@ -336,7 +336,12 @@ class HiveCompatibilitySuite extends HiveQueryFileTest 
with BeforeAndAfter {
     "udf_format_number",
     "udf_round",
     "udf_round_3",
-    "view_cast"
+    "view_cast",
+
+    // These tests check the VIEW table definition, but Spark handles CREATE 
VIEW itself and
+    // generates different View Expanded Text.
+    "alter_view_as_select",
+    "show_create_table_view"
   )
 
   /**
@@ -361,7 +366,6 @@ class HiveCompatibilitySuite extends HiveQueryFileTest with 
BeforeAndAfter {
     "alter_table_serde",
     "alter_varchar1",
     "alter_varchar2",
-    "alter_view_as_select",
     "ambiguous_col",
     "annotate_stats_join",
     "annotate_stats_limit",
@@ -833,7 +837,6 @@ class HiveCompatibilitySuite extends HiveQueryFileTest with 
BeforeAndAfter {
     "show_create_table_index",
     "show_create_table_partitioned",
     "show_create_table_serde",
-    "show_create_table_view",
     "show_describe_func_quotes",
     "show_functions",
     "show_partitions",


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to