> On Sept. 3, 2015, 1:34 a.m., Xuefu Zhang wrote: > > beeline/src/java/org/apache/hive/beeline/BeeLine.java, line 1094 > > <https://reviews.apache.org/r/38044/diff/1/?file=1062103#file1062103line1094> > > > > with line 1081-1085, is this still possible? I understand this is just > > moving code around.
That's possible. You can deep into the code of AbstractCommandHandler. The condition of match is whether the names is starting with the first part of the line. It means the command "!l;" is valid since there's only one match "list" exists. - cheng ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38044/#review97494 ----------------------------------------------------------- On Sept. 2, 2015, 3:16 p.m., cheng xu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/38044/ > ----------------------------------------------------------- > > (Updated Sept. 2, 2015, 3:16 p.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 > >
