hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "cheng xu" <cheng.a...@intel.com>
Subject Re: Review Request 27566: HIVE-8609: move beeline to jline2
Date Thu, 13 Nov 2014 03:28:00 GMT


> On Nov. 13, 2014, 2:06 a.m., Mohit Sabharwal wrote:
> > beeline/src/java/org/apache/hive/beeline/BeeLineCommandCompleter.java, line 35
> > <https://reviews.apache.org/r/27566/diff/4/?file=759115#file759115line35>
> >
> >     nit: add @Override

The override keyword is not applicable here since it's newly created in this class.


> On Nov. 13, 2014, 2:06 a.m., Mohit Sabharwal wrote:
> > hcatalog/hcatalog-pig-adapter/pom.xml, line 59
> > <https://reviews.apache.org/r/27566/diff/4/?file=759132#file759132line59>
> >
> >     2.12 ?

I am afraid not, since pig is using old versioned jline. To pass the UT, we have make it to
0.9.94.


- cheng


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27566/#review60811
-----------------------------------------------------------


On Nov. 12, 2014, 2:43 a.m., cheng xu wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/27566/
> -----------------------------------------------------------
> 
> (Updated Nov. 12, 2014, 2:43 a.m.)
> 
> 
> Review request for hive.
> 
> 
> Repository: hive-git
> 
> 
> Description
> -------
> 
> HIVE-8609: move beeline to jline2
> The following will be changed:
> * MultiCompletor-> AggregateCompleter
> * SimpleCompletor->StringsCompleter
> * Terminal.getTerminalWidth() -> Terminal.getWidth()
> * Terminal is an interface now; -> use TerminalFactory to get instances of a Terminal
> * String -> CharSequence
> 
> 
> Diffs
> -----
> 
>   LICENSE 2885945 
>   beeline/src/java/org/apache/hive/beeline/AbstractCommandHandler.java a9479d5 
>   beeline/src/java/org/apache/hive/beeline/BeeLine.java 8539a41 
>   beeline/src/java/org/apache/hive/beeline/BeeLineCommandCompleter.java PRE-CREATION

>   beeline/src/java/org/apache/hive/beeline/BeeLineCommandCompletor.java 52313e6 
>   beeline/src/java/org/apache/hive/beeline/BeeLineCompleter.java PRE-CREATION 
>   beeline/src/java/org/apache/hive/beeline/BeeLineCompletor.java c6bb4fe 
>   beeline/src/java/org/apache/hive/beeline/BeeLineOpts.java f73fb44 
>   beeline/src/java/org/apache/hive/beeline/BooleanCompleter.java PRE-CREATION 
>   beeline/src/java/org/apache/hive/beeline/BooleanCompletor.java 3e88c53 
>   beeline/src/java/org/apache/hive/beeline/ClassNameCompleter.java PRE-CREATION 
>   beeline/src/java/org/apache/hive/beeline/CommandHandler.java bab1778 
>   beeline/src/java/org/apache/hive/beeline/Commands.java 7e366dc 
>   beeline/src/java/org/apache/hive/beeline/DatabaseConnection.java ab67700 
>   beeline/src/java/org/apache/hive/beeline/ReflectiveCommandHandler.java 2b957f2 
>   beeline/src/java/org/apache/hive/beeline/SQLCompleter.java PRE-CREATION 
>   beeline/src/java/org/apache/hive/beeline/SQLCompletor.java 844b9ae 
>   beeline/src/java/org/apache/hive/beeline/TableNameCompletor.java bc0d9be 
>   cli/src/java/org/apache/hadoop/hive/cli/CliDriver.java 0ccaacb 
>   cli/src/test/org/apache/hadoop/hive/cli/TestCliDriverMethods.java 88a37d5 
>   hcatalog/hcatalog-pig-adapter/pom.xml 2d959e6 
>   pom.xml ec8c4fe 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/tez/TezJobMonitor.java dea3460 
> 
> Diff: https://reviews.apache.org/r/27566/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> cheng xu
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message