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

Eugene Koifman commented on HIVE-12439:
---------------------------------------

TxnHandler.timeoutLocks() has
"          LOG.info("Deleted " + rc + " ext locks from HIVE_LOCKS due to 
timeout (vs. " +
              extLockIDs.size() + " found. List: " + extLockIDs + ") 
maxHeartbeatTime=" + maxHeartbeatTime);
"

It's meaning is now different.  It used to be printed at the very end so that 
"rc" was total count of deleted locks.  Now it won't make sense any more since 
rc is per query, extLockIDs.size is per timeOutLocks()

otherwise looks good

> CompactionTxnHandler.markCleaned() and TxnHandler.openTxns() misc improvements
> ------------------------------------------------------------------------------
>
>                 Key: HIVE-12439
>                 URL: https://issues.apache.org/jira/browse/HIVE-12439
>             Project: Hive
>          Issue Type: Improvement
>          Components: Metastore, Transactions
>    Affects Versions: 1.0.0
>            Reporter: Eugene Koifman
>            Assignee: Wei Zheng
>         Attachments: HIVE-12439.1.patch, HIVE-12439.2.patch
>
>
> # add a safeguard to make sure IN clause is not too large; break up by txn id 
> to delete from TXN_COMPONENTS where tc_txnid in ...
> # TxnHandler. openTxns() - use 1 insert with many rows in values() clause, 
> rather than 1 DB roundtrip per row



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to