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

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


Bugs: HIVE-14391
    https://issues.apache.org/jira/browse/HIVE-14391


Repository: hive-git


Description
-------

The aim of this patch, to enable the Accumulo qtests to run again.

It contains the following:
- AccumuloQTestUtil change, so the configuration changes are set not only in 
init(), but in clearTestSideEffects() as well
- AccumuloTestSetup changes:
   - Removing temporary
   - Configuration update
- 3 qtest golden output rewrites - here I am a little nervous, since I am not 
sure that all of the change are caused by other changes in Hive - I would be 
happy if someone with more experience can confirm it.
- master-mr2.properties change to run the tests


Diffs
-----

  accumulo-handler/src/test/results/positive/accumulo_predicate_pushdown.q.out 
eb03041 
  accumulo-handler/src/test/results/positive/accumulo_queries.q.out d3c717b 
  
accumulo-handler/src/test/results/positive/accumulo_single_sourced_multi_insert.q.out
 07af080 
  itests/qtest-accumulo/pom.xml a4a5c75 
  
itests/util/src/main/java/org/apache/hadoop/hive/accumulo/AccumuloQTestUtil.java
 88bc0bc 
  
itests/util/src/main/java/org/apache/hadoop/hive/accumulo/AccumuloTestSetup.java
 73d5f15 
  
itests/util/src/main/java/org/apache/hadoop/hive/cli/control/CoreAccumuloCliDriver.java
 a5d2711 
  testutils/ptest2/conf/deployed/master-mr2.properties 02a560e 

Diff: https://reviews.apache.org/r/52984/diff/


Testing
-------

Run the test on my computer several times in several combination


Thanks,

Peter Vary

Reply via email to