> On Sept. 13, 2018, 8:03 a.m., Prasanth_J wrote: > > ql/src/java/org/apache/hadoop/hive/ql/exec/tez/TezExternalSessionState.java > > Lines 72 (patched) > > <https://reviews.apache.org/r/68709/diff/1/?file=2088718#file2088718line72> > > > > These resources may not be required. External sessions are assumed to > > have all the resources that it requires.
It's used in Tez object construction... I think Eric told me it was important for the reconnect patch to still add the resources. So for now I'll keep this in. > On Sept. 13, 2018, 8:03 a.m., Prasanth_J wrote: > > ql/src/java/org/apache/hadoop/hive/ql/exec/tez/TezExternalSessionsRegistryClient.java > > Lines 37 (patched) > > <https://reviews.apache.org/r/68709/diff/1/?file=2088719#file2088719line37> > > > > Yes. Tez needs severals fixes in addition to registry client > > 1) One that cuts of yarn/RM interaction > > 2) Registry configs via TezConfiguration > > 3) Handling DAG recovery for external sessions can you file some Tez jiras for that? - Sergey ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68709/#review208584 ----------------------------------------------------------- On Sept. 13, 2018, 2:19 a.m., Sergey Shelukhin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/68709/ > ----------------------------------------------------------- > > (Updated Sept. 13, 2018, 2:19 a.m.) > > > Review request for hive and Prasanth_J. > > > Repository: hive-git > > > Description > ------- > > see jira > > > Diffs > ----- > > common/src/java/org/apache/hadoop/hive/conf/HiveConf.java aa58d7445c > itests/util/src/main/java/org/apache/hadoop/hive/ql/QTestUtil.java > bfa3d5d7d2 > ql/pom.xml a55cbe380d > > ql/src/java/org/apache/hadoop/hive/ql/exec/tez/AbstractTriggerValidator.java > PRE-CREATION > > ql/src/java/org/apache/hadoop/hive/ql/exec/tez/KillTriggerActionHandler.java > f357775c86 > ql/src/java/org/apache/hadoop/hive/ql/exec/tez/TezExternalSessionState.java > PRE-CREATION > > ql/src/java/org/apache/hadoop/hive/ql/exec/tez/TezExternalSessionsRegistryClient.java > PRE-CREATION > ql/src/java/org/apache/hadoop/hive/ql/exec/tez/TezSession.java PRE-CREATION > ql/src/java/org/apache/hadoop/hive/ql/exec/tez/TezSessionPool.java > 89954cba67 > ql/src/java/org/apache/hadoop/hive/ql/exec/tez/TezSessionPoolManager.java > 2633390861 > ql/src/java/org/apache/hadoop/hive/ql/exec/tez/TezSessionPoolSession.java > d3748edb86 > ql/src/java/org/apache/hadoop/hive/ql/exec/tez/TezSessionState.java > 08e65a4a6d > ql/src/java/org/apache/hadoop/hive/ql/exec/tez/TezTask.java f2ed07add5 > > ql/src/java/org/apache/hadoop/hive/ql/exec/tez/TriggerValidatorRunnable.java > 670184b0ac > ql/src/java/org/apache/hadoop/hive/ql/exec/tez/WmTezSession.java fa2b02e591 > ql/src/java/org/apache/hadoop/hive/ql/exec/tez/WorkloadManager.java > 5326e3590f > > ql/src/java/org/apache/hadoop/hive/ql/exec/tez/WorkloadManagerFederation.java > 4b5022a101 > ql/src/java/org/apache/hadoop/hive/ql/session/SessionState.java 71e130b608 > ql/src/java/org/apache/hadoop/hive/ql/wm/SessionTriggerProvider.java > 16106f481b > ql/src/test/org/apache/hadoop/hive/ql/exec/tez/SampleTezSessionState.java > f5ab981f26 > ql/src/test/org/apache/hadoop/hive/ql/exec/tez/TestTezSessionPool.java > d5b683f788 > ql/src/test/org/apache/hadoop/hive/ql/exec/tez/TestTezTask.java b67aec371d > ql/src/test/org/apache/hadoop/hive/ql/exec/tez/TestWorkloadManager.java > 4659ecb97b > > > Diff: https://reviews.apache.org/r/68709/diff/1/ > > > Testing > ------- > > > Thanks, > > Sergey Shelukhin > >