----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64356/#review192968 -----------------------------------------------------------
ql/src/java/org/apache/hadoop/hive/ql/exec/tez/WorkloadManager.java Lines 1816 (patched) <https://reviews.apache.org/r/64356/#comment271398> nit: use try..catch..finally instead of isOk? - Prasanth_J On Dec. 5, 2017, 11:38 p.m., Sergey Shelukhin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/64356/ > ----------------------------------------------------------- > > (Updated Dec. 5, 2017, 11:38 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/WorkloadManager.java > ecdcf1251002e47bf4d31ec3a4b80c4404865be4 > ql/src/test/org/apache/hadoop/hive/ql/exec/tez/TestWorkloadManager.java > 78df962a3af8c57d7bbefea1a301c9ddbe281ac9 > > > Diff: https://reviews.apache.org/r/64356/diff/1/ > > > Testing > ------- > > > Thanks, > > Sergey Shelukhin > >