[ 
https://issues.apache.org/jira/browse/HIVE-16880?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16084108#comment-16084108
 ] 

Daniel Voros commented on HIVE-16880:
-------------------------------------

[~belugabehr] are you sure that using immutable lists won't affect the behavior 
in these cases? For example 
[here|https://github.com/apache/hive/commit/8f004997025f032242a5b2db4c6baf9256e0ecbd#diff-6b5f3b952d1387946d488fc2d4432ee1R1228]
 in the constructor of AggrStats the list will be stored as a field and we 
might try to add to it later.

> Remove ArrayList Instantiation For Empty Arrays
> -----------------------------------------------
>
>                 Key: HIVE-16880
>                 URL: https://issues.apache.org/jira/browse/HIVE-16880
>             Project: Hive
>          Issue Type: Improvement
>          Components: Metastore
>    Affects Versions: 2.1.1, 3.0.0
>            Reporter: BELUGA BEHR
>            Assignee: BELUGA BEHR
>            Priority: Trivial
>             Fix For: 3.0.0
>
>         Attachments: HIVE-16880.1.patch, HIVE-16880.2.patch
>
>
> Class {{org.apache.hadoop.hive.metastore.MetaStoreDirectSql}} uses a lot of 
> empty arrays in the code.  Please replace with a static empty array instead 
> of all the instantiation.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to