> On May 14, 2020, 4:31 a.m., Ashutosh Chauhan wrote: > > llap-server/src/java/org/apache/hadoop/hive/llap/daemon/impl/ContainerRunnerImpl.java > > Lines 638 (patched) > > <https://reviews.apache.org/r/72499/diff/1/?file=2231454#file2231454line639> > > > > Bound this queue by number of executors?
Will fix this in next patch. - Rajesh ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72499/#review220748 ----------------------------------------------------------- On May 12, 2020, 12:06 p.m., Rajesh Balamohan wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/72499/ > ----------------------------------------------------------- > > (Updated May 12, 2020, 12:06 p.m.) > > > Review request for hive, Ashutosh Chauhan and Gopal V. > > > Bugs: HIVE-23446 > https://issues.apache.org/jira/browse/HIVE-23446 > > > Repository: hive-git > > > Description > ------- > > Currently UGI pool is maintained at QueryInfo level. However, when short > queries and lots of AMs are there, it ends missing IPC connection cache. Too > many connections are are also established. Patch tries to avoid that by > maintaining this at ContainerRunner level. It retains the current behaviour > of having multiple connection to same AM (otherwise can get bottlenecked on > single connection) > > > Diffs > ----- > > > llap-server/src/java/org/apache/hadoop/hive/llap/daemon/impl/ContainerRunnerImpl.java > 6a13b55e69 > llap-server/src/java/org/apache/hadoop/hive/llap/daemon/impl/QueryInfo.java > 00fed15d2b > > llap-server/src/java/org/apache/hadoop/hive/llap/daemon/impl/QueryTracker.java > eae8e08540 > > llap-server/src/test/org/apache/hadoop/hive/llap/daemon/impl/TaskExecutorTestHelpers.java > 50dec4759e > > > Diff: https://reviews.apache.org/r/72499/diff/1/ > > > Testing > ------- > > > Thanks, > > Rajesh Balamohan > >
