[ 
https://issues.apache.org/jira/browse/HIVE-23846?focusedWorklogId=461766&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-461766
 ]

ASF GitHub Bot logged work on HIVE-23846:
-----------------------------------------

                Author: ASF GitHub Bot
            Created on: 21/Jul/20 22:13
            Start Date: 21/Jul/20 22:13
    Worklog Time Spent: 10m 
      Work Description: hsnusonic closed pull request #1266:
URL: https://github.com/apache/hive/pull/1266


   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Issue Time Tracking
-------------------

    Worklog Id:     (was: 461766)
    Time Spent: 40m  (was: 0.5h)

> Avoid unnecessary serialization and deserialization of bitvectors
> -----------------------------------------------------------------
>
>                 Key: HIVE-23846
>                 URL: https://issues.apache.org/jira/browse/HIVE-23846
>             Project: Hive
>          Issue Type: Bug
>          Components: Standalone Metastore
>            Reporter: Yu-Wen Lai
>            Assignee: Yu-Wen Lai
>            Priority: Major
>              Labels: pull-request-available
>             Fix For: 4.0.0
>
>          Time Spent: 40m
>  Remaining Estimate: 0h
>
> In the method *getNdvEstimator* of <datatype>*ColumnStatsDataInspector*, it 
> will call isSetBitVectors(), in which it serializes the bitvectors again when 
> we already have deserialized bitvectors _ndvEstimator_. For example, we can 
> see this pattern from 
> [LongColumnStatsDataInspector|[https://github.com/apache/hive/blob/master/standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/columnstats/cache/LongColumnStatsDataInspector.java#L106]].
> This method could check if the _ndvEstimator_ is set first so that it won't 
> need to serialize and deserialize back.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to