You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@hive.apache.org by Alan Gates <ga...@hortonworks.com> on 2014/09/04 18:50:56 UTC

Review Request 25343: HIVE-7899 txnMgr should be session specific

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25343/
-----------------------------------------------------------

Review request for hive and Ashutosh Chauhan.


Bugs: HIVE-7899
    https://issues.apache.org/jira/browse/HIVE-7899


Repository: hive-git


Description
-------

This patch moves the TxnManager instance from Driver to SessionState, since multiple queries can share a single session, and it is convenient in other parts of the code to be able to get to the transaction manager.  It also stores the current transaction id and whether we are in autocommit in SessionState.


Diffs
-----

  ql/src/java/org/apache/hadoop/hive/ql/Driver.java 0533ae8 
  ql/src/java/org/apache/hadoop/hive/ql/session/SessionState.java df66f83 

Diff: https://reviews.apache.org/r/25343/diff/


Testing
-------

Ran existing transaction manager tests.


Thanks,

Alan Gates