----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65579/#review197150 -----------------------------------------------------------
Fix it, then Ship it! Comment on unrelated error. Changes are fine. ql/src/test/results/clientpositive/llap/resourceplan.q.out Lines 1545 (patched) <https://reviews.apache.org/r/65579/#comment277275> Is this expected? I know its not related to the code changes, but this might supress someother error. - Harish Jaiprakash On Feb. 9, 2018, 4:36 a.m., Sergey Shelukhin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/65579/ > ----------------------------------------------------------- > > (Updated Feb. 9, 2018, 4:36 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/parse/DDLSemanticAnalyzer.java > b766791ebc > ql/src/test/queries/clientpositive/resourceplan.q 7314585415 > ql/src/test/results/clientpositive/llap/resourceplan.q.out b23720d1a8 > > > Diff: https://reviews.apache.org/r/65579/diff/1/ > > > Testing > ------- > > > Thanks, > > Sergey Shelukhin > >