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

Steve Yeom commented on HIVE-19931:
-----------------------------------

The one line incorrect code caused wrong results:
+++ 
standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/ObjectStore.java
@@ -4094,7 +4094,7 @@ public void alterTable(String catName, String dbname, 
String name, Table newTabl
       if (newTable.getValidWriteIdList() != null &&
           TxnUtils.isTransactionalTable(newTable)) {
         // Check concurrent INSERT case and set false to the flag.
-        if (isCurrentStatsValidForTheQuery(oldt, newt.getTxnId(), 
newt.getWriteIdList(),
+        if (!isCurrentStatsValidForTheQuery(oldt, newt.getTxnId(), 
newt.getWriteIdList(),

I will add this patch to the patch for HIVE-19931

> stats_nonpart.q test run shows possibly wrong results.
> ------------------------------------------------------
>
>                 Key: HIVE-19931
>                 URL: https://issues.apache.org/jira/browse/HIVE-19931
>             Project: Hive
>          Issue Type: Sub-task
>          Components: Transactions
>    Affects Versions: 4.0.0
>            Reporter: Steve Yeom
>            Assignee: Steve Yeom
>            Priority: Major
>             Fix For: 4.0.0
>
>




--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to