[hive] branch master updated: HIVE-22631: Avoid deep copying partition list in listPartitionsByExpr (Vineet Garg, reviewed by Jesus Camacho Rodriguez)

[vg...@apache.org]

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

vgarg pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/hive.git


The following commit(s) were added to refs/heads/master by this push:
     new 1f541b5  HIVE-22631: Avoid deep copying partition list in listPartitionsByExpr (Vineet Garg, reviewed by Jesus Camacho Rodriguez)
1f541b5 is described below

commit 1f541b52314519e206dcb0f1aa2ebceb8e44d7a6
Author: Vineet Garg <vg...@apache.org>
AuthorDate: Tue Jan 14 12:06:21 2020 +0530

    HIVE-22631: Avoid deep copying partition list in listPartitionsByExpr (Vineet Garg, reviewed by Jesus Camacho Rodriguez)
---
 .../java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java     | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/standalone-metastore/metastore-common/src/main/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java b/standalone-metastore/metastore-common/src/main/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java
index 736c2b0..cc97418 100644
--- a/standalone-metastore/metastore-common/src/main/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java
+++ b/standalone-metastore/metastore-common/src/main/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java
@@ -1936,7 +1936,8 @@ public class HiveMetaStoreClient implements IMetaStoreClient, AutoCloseable {
 
     r.setPartitions(FilterUtils.filterPartitionsIfEnabled(isClientFilterEnabled, filterHook, r.getPartitions()));
     // TODO: in these methods, do we really need to deepcopy?
-    deepCopyPartitions(r.getPartitions(), result);
+    //deepCopyPartitions(r.getPartitions(), result);
+    result.addAll(r.getPartitions());
     return !r.isSetHasUnknownPartitions() || r.isHasUnknownPartitions(); // Assume the worst.
   }