hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Peter Vary <pv...@cloudera.com>
Subject Review Request 52978: HIVE-14459 TestBeeLineDriver - migration and re-enable
Date Tue, 18 Oct 2016 07:02:38 GMT

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

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