Prasanth Jayachandran commented on HIVE-19578:

Actually, let me put up another patch that retains temp list. getSize() does 
not have to be precise. We can return sparseMap.size() + tempListIdx which is 
sufficient enough. I tried this yesterday and for some reason few tests failed. 
I figured out what went wrong. 


> HLL merges tempList on every add
> --------------------------------
>                 Key: HIVE-19578
>                 URL: https://issues.apache.org/jira/browse/HIVE-19578
>             Project: Hive
>          Issue Type: Bug
>            Reporter: Sergey Shelukhin
>            Assignee: Prasanth Jayachandran
>            Priority: Major
>         Attachments: HIVE-19578.1.patch, Screen Shot 2018-05-16 at 15.29.12 
> .png
>  See comments on HIVE-18866; this has significant perf overhead after the 
> even bigger overhead from hashing is removed.  !Screen Shot 2018-05-16 at 
> 15.29.12 .png!

This message was sent by Atlassian JIRA

Reply via email to