[ https://issues.apache.org/jira/browse/HIVE-19820?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16543612#comment-16543612 ]
Sergey Shelukhin commented on HIVE-19820: ----------------------------------------- There are two problems with aggregation - first write ID is not being passed in, but more importantly merge needs different handling than the standard path - if we fail to get stats it should set stats to invalid instead of blindly writing unmerged stats. So, the check cannot just be "get valid stats", it should "get any stats", then check validity. > add ACID stats support to background stats updater and fix bunch of edge > cases found in SU tests > ------------------------------------------------------------------------------------------------ > > Key: HIVE-19820 > URL: https://issues.apache.org/jira/browse/HIVE-19820 > Project: Hive > Issue Type: Sub-task > Reporter: Sergey Shelukhin > Assignee: Sergey Shelukhin > Priority: Major > Attachments: HIVE-19820.01-master-txnstats.patch, > HIVE-19820.01.patch, HIVE-19820.02-master-txnstats.patch, > HIVE-19820.03-master-txnstats.patch, HIVE-19820.03.patch, > HIVE-19820.04-master-txnstats.patch, HIVE-19820.patch, > branch-19820.02.nogen.patch, branch-19820.03.nogen.patch, > branch-19820.nogen.patch, branch-19820.nogen.patch > > > Follow-up from HIVE-19418. > Right now it checks whether stats are valid in an old-fashioned way... and > also gets ACID state, and discards it without using. > When ACID stats are implemented, ACID state needs to be used to do > version-aware valid stats checks. -- This message was sent by Atlassian JIRA (v7.6.3#76005)