hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Peter Vary <>
Subject Re: Review Request 52978: HIVE-14459 TestBeeLineDriver - migration and re-enable
Date Tue, 28 Feb 2017 10:28:01 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated Feb. 28, 2017, 10:28 a.m.)

Review request for hive, Zoltan Haindrich, Marta Kuczora, Miklos Csanady, Prasanth_J, Sergey
Shelukhin, Sergio Pena, Siddharth Seth, and Barna Zsombor Klara.


- cleanupLocalDirOnStartup is true
- Added the script directory the the init scripts, so they should be running now

Bugs: HIVE-14459

Repository: hive-git


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 (updated)

  beeline/src/java/org/apache/hive/beeline/util/ 81f1b0e 
  itests/qtest/src/test/java/org/apache/hadoop/hive/cli/ cb276e6

  itests/src/test/resources/ d344464 
  itests/util/pom.xml 6d93dc1 
  itests/util/src/main/java/org/apache/hadoop/hive/cli/control/ af8ec67 
  itests/util/src/main/java/org/apache/hadoop/hive/cli/control/ e5144e3

  ql/src/test/results/clientpositive/beeline/escape_comments.q.out PRE-CREATION 



Manually on my computer several runs.
Waiting for the QA tests


Peter Vary

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