hive-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ser...@apache.org
Subject [51/61] [abbrv] hive git commit: HIVE-17731 : add a backward compat option for external users to HIVE-11985 (Sergey Shelukhin, reviewed by Jesus Camacho Rodriguez)
Date Tue, 10 Oct 2017 00:52:25 GMT
HIVE-17731 : add a backward compat option for external users to HIVE-11985 (Sergey Shelukhin,
reviewed by Jesus Camacho Rodriguez)


Project: http://git-wip-us.apache.org/repos/asf/hive/repo
Commit: http://git-wip-us.apache.org/repos/asf/hive/commit/8c50fd7d
Tree: http://git-wip-us.apache.org/repos/asf/hive/tree/8c50fd7d
Diff: http://git-wip-us.apache.org/repos/asf/hive/diff/8c50fd7d

Branch: refs/heads/hive-14535
Commit: 8c50fd7d4bd9714a3120fde47a82f287bf7f3581
Parents: 2fdd5af
Author: sergey <sershe@apache.org>
Authored: Mon Oct 9 12:34:33 2017 -0700
Committer: sergey <sershe@apache.org>
Committed: Mon Oct 9 12:34:33 2017 -0700

----------------------------------------------------------------------
 common/src/java/org/apache/hadoop/hive/conf/HiveConf.java | 6 ++++++
 ql/src/java/org/apache/hadoop/hive/ql/metadata/Table.java | 1 +
 2 files changed, 7 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hive/blob/8c50fd7d/common/src/java/org/apache/hadoop/hive/conf/HiveConf.java
----------------------------------------------------------------------
diff --git a/common/src/java/org/apache/hadoop/hive/conf/HiveConf.java b/common/src/java/org/apache/hadoop/hive/conf/HiveConf.java
index e3aa71f3..3025692 100644
--- a/common/src/java/org/apache/hadoop/hive/conf/HiveConf.java
+++ b/common/src/java/org/apache/hadoop/hive/conf/HiveConf.java
@@ -1228,6 +1228,12 @@ public class HiveConf extends Configuration {
         "org.apache.hadoop.hive.serde2.lazybinary.LazyBinarySerDe",
         "SerDes retrieving schema from metastore. This is an internal parameter."),
 
+    @Deprecated
+    HIVE_LEGACY_SCHEMA_FOR_ALL_SERDES("hive.legacy.schema.for.all.serdes",
+        false,
+        "A backward compatibility setting for external metastore users that do not handle
\n" +
+        SERDESUSINGMETASTOREFORSCHEMA.varname + " correctly. This may be removed at any time."),
+
     HIVEHISTORYFILELOC("hive.querylog.location",
         "${system:java.io.tmpdir}" + File.separator + "${system:user.name}",
         "Location of Hive run time structured log file"),

http://git-wip-us.apache.org/repos/asf/hive/blob/8c50fd7d/ql/src/java/org/apache/hadoop/hive/ql/metadata/Table.java
----------------------------------------------------------------------
diff --git a/ql/src/java/org/apache/hadoop/hive/ql/metadata/Table.java b/ql/src/java/org/apache/hadoop/hive/ql/metadata/Table.java
index 43e202c..cdc0561 100644
--- a/ql/src/java/org/apache/hadoop/hive/ql/metadata/Table.java
+++ b/ql/src/java/org/apache/hadoop/hive/ql/metadata/Table.java
@@ -967,6 +967,7 @@ public class Table implements Serializable {
   public static boolean shouldStoreFieldsInMetastore(
       HiveConf conf, String serdeLib, Map<String, String> tableParams) {
     if (hasMetastoreBasedSchema(conf, serdeLib))  return true;
+    if (HiveConf.getBoolVar(conf, ConfVars.HIVE_LEGACY_SCHEMA_FOR_ALL_SERDES)) return true;
     // Table may or may not be using metastore. Only the SerDe can tell us.
     AbstractSerDe deserializer = null;
     try {


Mime
View raw message