----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11334/#review25859 -----------------------------------------------------------
beeline/src/java/org/apache/hive/beeline/BeeLine.java <https://reviews.apache.org/r/11334/#comment50420> thats fine beeline/src/java/org/apache/hive/beeline/BeeLine.properties <https://reviews.apache.org/r/11334/#comment50421> It is more accurate to say that this is a hive specific variable, as beeline is still a generic tool. beeline/src/java/org/apache/hive/beeline/BeeLineOpts.java <https://reviews.apache.org/r/11334/#comment50422> Yes, makes sense to call this hivevariables itself. beeline/src/java/org/apache/hive/beeline/DatabaseConnection.java <https://reviews.apache.org/r/11334/#comment50423> This change has not been made in revised patch. - Thejas Nair On Aug. 24, 2013, 8:19 p.m., Xuefu Zhang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/11334/ > ----------------------------------------------------------- > > (Updated Aug. 24, 2013, 8:19 p.m.) > > > Review request for hive and Ashutosh Chauhan. > > > Bugs: HIVE-4568 > https://issues.apache.org/jira/browse/HIVE-4568 > > > Repository: hive-git > > > Description > ------- > > 1. Added command variable substition > 2. Added test case > > > Diffs > ----- > > beeline/src/java/org/apache/hive/beeline/BeeLine.java 4c6eb9b > beeline/src/java/org/apache/hive/beeline/BeeLine.properties b6650cf > beeline/src/java/org/apache/hive/beeline/BeeLineOpts.java 61bdeee > beeline/src/java/org/apache/hive/beeline/DatabaseConnection.java c70003d > beeline/src/test/org/apache/hive/beeline/src/test/TestBeeLineWithArgs.java > 030f6b0 > > Diff: https://reviews.apache.org/r/11334/diff/ > > > Testing > ------- > > > Thanks, > > Xuefu Zhang > >