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

[vgarg]

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.
   }