> On Nov. 9, 2017, 2:05 a.m., Prasanth_J wrote: > > ql/src/java/org/apache/hadoop/hive/ql/exec/tez/WorkloadManager.java > > Lines 494 (patched) > > <https://reviews.apache.org/r/63533/diff/3/?file=1885126#file1885126line506> > > > > why are we putting back if NOT_FOUND? should this be destroyed instead? > > this session will still be NOT_FOUND in next iteration of event processing > > right?
No, it closes the old session and opens a new one - Sergey ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63533/#review190530 ----------------------------------------------------------- On Nov. 8, 2017, 9:12 p.m., Sergey Shelukhin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/63533/ > ----------------------------------------------------------- > > (Updated Nov. 8, 2017, 9:12 p.m.) > > > Review request for hive and Prasanth_J. > > > Repository: hive-git > > > Description > ------- > > see jira > > > Diffs > ----- > > ql/src/java/org/apache/hadoop/hive/ql/exec/tez/TezSessionPool.java > fa0eedb1e2 > ql/src/java/org/apache/hadoop/hive/ql/exec/tez/WmTezSession.java 0dd1433a6c > ql/src/java/org/apache/hadoop/hive/ql/exec/tez/WorkloadManager.java > 169991c639 > ql/src/test/org/apache/hadoop/hive/ql/exec/tez/TestWorkloadManager.java > 0347e914bc > > > Diff: https://reviews.apache.org/r/63533/diff/3/ > > > Testing > ------- > > > Thanks, > > Sergey Shelukhin > >
