hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Zoltan Haindrich <k...@rxd.hu>
Subject Re: Review Request 52978: HIVE-14459 TestBeeLineDriver - migration and re-enable
Date Tue, 18 Oct 2016 07:21:16 GMT

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


Ship it!




Hello Peter!

Beeline tests are coming back into business? That's great! ;)
I think its a good idea to add just one case for now - that way we can ensure that its in
working order; its a big step forward from not running any of them.

cheers,
Zoltan

- Zoltan Haindrich


On Oct. 18, 2016, 7:02 a.m., Peter Vary wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/52978/
> -----------------------------------------------------------
> 
> (Updated Oct. 18, 2016, 7:02 a.m.)
> 
> 
> Review request for hive, Zoltan Haindrich, Marta Kuczora, Miklos Csanady, Prasanth_J,
Sergey Shelukhin, Sergio Pena, Siddharth Seth, and Barna Zsombor Klara.
> 
> 
> Bugs: HIVE-14459
>     https://issues.apache.org/jira/browse/HIVE-14459
> 
> 
> Repository: hive-git
> 
> 
> Description
> -------
> 
> Kept the changes minimal with the sole goal to be able to run the BeeLine query tests
multiple times successfully.
> - Enabled the driver
> - Modified the regexps to hide when comparing the results
> - Configured to run only 1 qtest file - so we can test, and could decide later of the
beeline testing scope
> - Added required dependencies to pom
> - Added specific results dir for beeline q.out-s
> 
> 
> Diffs
> -----
> 
>   beeline/src/java/org/apache/hive/beeline/util/QFileClient.java 81f1b0e 
>   itests/qtest/src/test/java/org/apache/hadoop/hive/cli/DisabledTestBeeLineDriver.java
cb276e6 
>   itests/qtest/src/test/java/org/apache/hadoop/hive/cli/TestBeeLineDriver.java PRE-CREATION

>   itests/src/test/resources/testconfiguration.properties 97e310d 
>   itests/util/pom.xml 6d93dc1 
>   itests/util/src/main/java/org/apache/hadoop/hive/cli/control/CliConfigs.java 366c7b4

>   itests/util/src/main/java/org/apache/hadoop/hive/cli/control/CoreBeeLineDriver.java
e5144e3 
>   ql/src/test/results/clientpositive/beeline/escape_comments.q.out PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/52978/diff/
> 
> 
> Testing
> -------
> 
> Manually on my computer several runs.
> Waiting for the QA tests
> 
> 
> Thanks,
> 
> Peter Vary
> 
>


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