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

Sushanth Sowmyan commented on HIVE-10521:
-----------------------------------------

Approved for 1.2 pending tests, Eugene's review & commit to master.

I've added it to the list here : 
https://cwiki.apache.org/confluence/display/Hive/Hive+1.2+Release+Status

> TxnHandler.timeOutTxns only times out some of the expired transactions
> ----------------------------------------------------------------------
>
>                 Key: HIVE-10521
>                 URL: https://issues.apache.org/jira/browse/HIVE-10521
>             Project: Hive
>          Issue Type: Bug
>          Components: Transactions
>    Affects Versions: 0.14.0, 1.0.0, 1.1.0
>            Reporter: Alan Gates
>            Assignee: Alan Gates
>         Attachments: HIVE-10521.patch
>
>
> {code}
>       for (int i = 0; i < 20 && rs.next(); i++) deadTxns.add(rs.getLong(1));
>       // We don't care whether all of the transactions get deleted or not,
>       // if some didn't it most likely means someone else deleted them in the 
> interum
>       if (deadTxns.size() > 0) abortTxns(dbConn, deadTxns);
> {code}
> While it makes sense to limit the number of transactions aborted in one pass 
> (since this get's translated to an IN clause) we should still make sure all 
> are timed out.  Also, 20 seems pretty small as a batch size.



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

Reply via email to