----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64595/#review193773 -----------------------------------------------------------
ql/src/java/org/apache/hadoop/hive/ql/parse/DDLSemanticAnalyzer.java Lines 1112 (patched) <https://reviews.apache.org/r/64595/#comment272422> We can use the API to add an invalid policy. Should we have this validation in the ObjectStore? Maybe the enum could be moved into the data classes like the resource plan status. - Harish Jaiprakash On Dec. 14, 2017, 9:09 a.m., Sergey Shelukhin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/64595/ > ----------------------------------------------------------- > > (Updated Dec. 14, 2017, 9:09 a.m.) > > > Review request for hive, Harish Jaiprakash and Prasanth_J. > > > Repository: hive-git > > > Description > ------- > > see jira > > > Diffs > ----- > > ql/src/java/org/apache/hadoop/hive/ql/exec/tez/WorkloadManager.java > dbdbbf25db > ql/src/java/org/apache/hadoop/hive/ql/parse/DDLSemanticAnalyzer.java > a09b7961c2 > ql/src/test/org/apache/hadoop/hive/ql/exec/tez/TestWorkloadManager.java > c58e4507f2 > ql/src/test/queries/clientpositive/resourceplan.q fc924a2f95 > ql/src/test/results/clientpositive/llap/resourceplan.q.out 7f3e784457 > > > Diff: https://reviews.apache.org/r/64595/diff/1/ > > > Testing > ------- > > > Thanks, > > Sergey Shelukhin > >
