----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17471/#review36039 -----------------------------------------------------------
ql/src/java/org/apache/hadoop/hive/ql/exec/tez/TezSessionPoolManager.java <https://reviews.apache.org/r/17471/#comment66866> can you remove trailing ws in the files. This shouldn't require a new run of the tests... service/src/java/org/apache/hive/service/server/HiveServer2.java <https://reviews.apache.org/r/17471/#comment66868> i believe this should be hidden behind conf check - Gunther Hagleitner On Feb. 28, 2014, 2:25 a.m., Vikram Dixit Kumaraswamy wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/17471/ > ----------------------------------------------------------- > > (Updated Feb. 28, 2014, 2:25 a.m.) > > > Review request for hive. > > > Bugs: HIVE-6325 > https://issues.apache.org/jira/browse/HIVE-6325 > > > Repository: hive-git > > > Description > ------- > > Enable using multiple concurrent sessions in tez. > > > Diffs > ----- > > common/src/java/org/apache/hadoop/hive/conf/HiveConf.java 6802b4d > conf/hive-default.xml.template 0b86b9c > ql/src/java/org/apache/hadoop/hive/ql/exec/tez/TezSessionPoolManager.java > PRE-CREATION > ql/src/java/org/apache/hadoop/hive/ql/exec/tez/TezSessionState.java b8552a3 > ql/src/java/org/apache/hadoop/hive/ql/exec/tez/TezTask.java bef5ba3 > ql/src/java/org/apache/hadoop/hive/ql/session/SessionState.java 16e9166 > ql/src/test/org/apache/hadoop/hive/ql/exec/tez/TestTezSessionPool.java > PRE-CREATION > ql/src/test/org/apache/hadoop/hive/ql/exec/tez/TestTezSessionState.java > PRE-CREATION > service/src/java/org/apache/hive/service/server/HiveServer2.java 8700c5b > > Diff: https://reviews.apache.org/r/17471/diff/ > > > Testing > ------- > > Added multi-threaded junit tests. > > > Thanks, > > Vikram Dixit Kumaraswamy > >