----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38044/#review97494 -----------------------------------------------------------
beeline/src/java/org/apache/hive/beeline/BeeLine.java (line 1091) <https://reviews.apache.org/r/38044/#comment153386> with line 1081-1085, is this still possible? I understand this is just moving code around. - Xuefu Zhang On Sept. 2, 2015, 7:16 a.m., cheng xu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/38044/ > ----------------------------------------------------------- > > (Updated Sept. 2, 2015, 7:16 a.m.) > > > Review request for hive and Xuefu Zhang. > > > Bugs: HIVE-11640 > https://issues.apache.org/jira/browse/HIVE-11640 > > > Repository: hive-git > > > Description > ------- > > Fixed issues includes: > * Support the case that one line contains multi-commands with ";" seperated > * Fix the nullempty string for new cli mode > * Resolve command doesn't support issue when executing files > > > Diffs > ----- > > beeline/src/java/org/apache/hive/beeline/BeeLine.java 1e4759b > beeline/src/java/org/apache/hive/beeline/Commands.java 5e5cfec > beeline/src/test/org/apache/hive/beeline/cli/TestHiveCli.java e06d2ea > > Diff: https://reviews.apache.org/r/38044/diff/ > > > Testing > ------- > > Local smoke test passed and unit test passed locally > > > Thanks, > > cheng xu > >
