[HIVE-10521] TxnHandler.timeOutTxns only times out some of the expired transactions

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 0.14.0, 1.0.0, 1.1.0
• Fix Version/s: 1.3.0, 2.0.0
• Component/s: Transactions
• Assignee: Alan Gates
• Reporter: Alan Gates
• Created: 28/Apr/15 18:23
• Updated: 01/Oct/19 22:07
• Resolved: 07/May/15 02:37

Description

      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);

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.