----------------------------------------------------------- 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