----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63381/#review189864 -----------------------------------------------------------
service/src/java/org/apache/hive/service/server/HiveServer2.java Line 140 (original), 142 (patched) <https://reviews.apache.org/r/63381/#comment267098> It was old code. When initialize default sessions is not set (or if queue names are empty), hasInitialSession will default to false and will get new session on every invocation of tezTask.getSession(). - Prasanth_J On Oct. 31, 2017, 7:27 p.m., Prasanth_J wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/63381/ > ----------------------------------------------------------- > > (Updated Oct. 31, 2017, 7:27 p.m.) > > > Review request for hive and Sergey Shelukhin. > > > Bugs: HIVE-17926 > https://issues.apache.org/jira/browse/HIVE-17926 > > > Repository: hive-git > > > Description > ------- > > HIVE-17926: Support triggers for non-pool sessions > > > Diffs > ----- > > > itests/hive-unit/src/test/java/org/apache/hive/jdbc/AbstractJdbcTriggersTest.java > PRE-CREATION > > itests/hive-unit/src/test/java/org/apache/hive/jdbc/TestTriggersNoTezSessionPool.java > PRE-CREATION > > itests/hive-unit/src/test/java/org/apache/hive/jdbc/TestTriggersTezSessionPoolManager.java > aeca663 > ql/src/java/org/apache/hadoop/hive/ql/exec/tez/TezSessionPool.java b67c933 > ql/src/java/org/apache/hadoop/hive/ql/exec/tez/TezSessionPoolManager.java > 9b4714f > service/src/java/org/apache/hive/service/server/HiveServer2.java 2c4fe7f > > > Diff: https://reviews.apache.org/r/63381/diff/4/ > > > Testing > ------- > > > Thanks, > > Prasanth_J > >
