----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17471/#review33962 -----------------------------------------------------------
ql/src/java/org/apache/hadoop/hive/ql/exec/tez/TezSessionPoolManager.java <https://reviews.apache.org/r/17471/#comment63814> Is TezSessionPoolManager going to be specific to HiveServer2? If yes, do you think it makes sense to have it extend org.apache.hive.service.CompositeService or org.apache.hive.service.AbstractService? - Vaibhav Gumashta On Feb. 6, 2014, 2:08 a.m., Vikram Dixit Kumaraswamy wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/17471/ > ----------------------------------------------------------- > > (Updated Feb. 6, 2014, 2:08 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 89c9349 > conf/hive-default.xml.template 420d959 > itests/util/src/main/java/org/apache/hadoop/hive/ql/QTestUtil.java 9b2e219 > 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 22a21c9 > ql/src/java/org/apache/hadoop/hive/ql/session/SessionState.java 64a8a60 > 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 > >
