You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hive.apache.org by sp...@apache.org on 2016/07/22 21:50:07 UTC

hive git commit: HIVE-14295: Some metastore event listeners always initialize deleteData as false (niklaus xiao, reviewed by Sergio Pena)

Repository: hive
Updated Branches:
  refs/heads/master d246aee62 -> 00bb58a39


HIVE-14295: Some metastore event listeners always initialize deleteData as false (niklaus xiao, reviewed by Sergio Pena)


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

Branch: refs/heads/master
Commit: 00bb58a390224e21b72e9380c2ef19a46645decb
Parents: d246aee
Author: niklaus xiao <st...@live.cn>
Authored: Fri Jul 22 16:49:33 2016 -0500
Committer: Sergio Pena <se...@cloudera.com>
Committed: Fri Jul 22 16:49:33 2016 -0500

----------------------------------------------------------------------
 .../org/apache/hadoop/hive/metastore/events/DropTableEvent.java    | 2 +-
 .../apache/hadoop/hive/metastore/events/PreDropPartitionEvent.java | 2 +-
 .../org/apache/hadoop/hive/metastore/events/PreDropTableEvent.java | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hive/blob/00bb58a3/metastore/src/java/org/apache/hadoop/hive/metastore/events/DropTableEvent.java
----------------------------------------------------------------------
diff --git a/metastore/src/java/org/apache/hadoop/hive/metastore/events/DropTableEvent.java b/metastore/src/java/org/apache/hadoop/hive/metastore/events/DropTableEvent.java
index 560b90a..34efa5f 100644
--- a/metastore/src/java/org/apache/hadoop/hive/metastore/events/DropTableEvent.java
+++ b/metastore/src/java/org/apache/hadoop/hive/metastore/events/DropTableEvent.java
@@ -31,7 +31,7 @@ public class DropTableEvent extends ListenerEvent {
     this.table = table;
     // In HiveMetaStore, the deleteData flag indicates whether DFS data should be
     // removed on a drop.
-    this.deleteData = false;
+    this.deleteData = deleteData;
   }
 
   /**

http://git-wip-us.apache.org/repos/asf/hive/blob/00bb58a3/metastore/src/java/org/apache/hadoop/hive/metastore/events/PreDropPartitionEvent.java
----------------------------------------------------------------------
diff --git a/metastore/src/java/org/apache/hadoop/hive/metastore/events/PreDropPartitionEvent.java b/metastore/src/java/org/apache/hadoop/hive/metastore/events/PreDropPartitionEvent.java
index ac391ee..456164f 100644
--- a/metastore/src/java/org/apache/hadoop/hive/metastore/events/PreDropPartitionEvent.java
+++ b/metastore/src/java/org/apache/hadoop/hive/metastore/events/PreDropPartitionEvent.java
@@ -36,7 +36,7 @@ public class PreDropPartitionEvent extends PreEventContext {
     super (PreEventType.DROP_PARTITION, handler);
     this.partitions = Collections.singletonList(partition);
     this.table = table;
-    this.deleteData = false;
+    this.deleteData = deleteData;
   }
 
   /**

http://git-wip-us.apache.org/repos/asf/hive/blob/00bb58a3/metastore/src/java/org/apache/hadoop/hive/metastore/events/PreDropTableEvent.java
----------------------------------------------------------------------
diff --git a/metastore/src/java/org/apache/hadoop/hive/metastore/events/PreDropTableEvent.java b/metastore/src/java/org/apache/hadoop/hive/metastore/events/PreDropTableEvent.java
index e804617..df22196 100644
--- a/metastore/src/java/org/apache/hadoop/hive/metastore/events/PreDropTableEvent.java
+++ b/metastore/src/java/org/apache/hadoop/hive/metastore/events/PreDropTableEvent.java
@@ -31,7 +31,7 @@ public class PreDropTableEvent extends PreEventContext {
     this.table = table;
     // In HiveMetaStore, the deleteData flag indicates whether DFS data should be
     // removed on a drop.
-    this.deleteData = false;
+    this.deleteData = deleteData;
   }
 
   /**