Home | About | Sematext search-lucene.com search-hadoop.com
 Search Hadoop and all its subprojects:

Switch to Threaded View
Hive >> mail # dev >> Review Request 12790: HIVE-4892: PTest2 cleanup after merge


Copy link to this message
-
Review Request 12790: HIVE-4892: PTest2 cleanup after merge

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

Review request for hive.
Bugs: HIVE-4892
    https://issues.apache.org/jira/browse/HIVE-4892
Repository: hive-git
Description
-------

Fixes the following issues:

README is out of date
Need to limit the number of failed source directories we copy back from the slaves
when looking for TEST-*.xml files we look at both the log directory (good) and the failed source directories (bad) therefore duplicating failures in jenkins report
We need to process bad hosts in the finally block of PTest.run (HIVE-4882)
Need a mechanism to clean the ivy and maven cache (HIVE-4882)
PTest2 fails to publish a comment to a JIRA sometimes (HIVE-4889)
Now that PTest2 is committed to the source tree it's copying in our TEST-SomeTest*.xml files
Diffs
-----

  testutils/ptest2/README.md a483851
  testutils/ptest2/src/main/java/org/apache/hive/ptest/api/client/PTestClient.java d6e9ce3
  testutils/ptest2/src/main/java/org/apache/hive/ptest/api/request/TestStartRequest.java b41726a
  testutils/ptest2/src/main/java/org/apache/hive/ptest/api/server/TestExecutor.java 9b2125c
  testutils/ptest2/src/main/java/org/apache/hive/ptest/execution/ExecutionPhase.java b5d7909
  testutils/ptest2/src/main/java/org/apache/hive/ptest/execution/HostExecutor.java 34a087c
  testutils/ptest2/src/main/java/org/apache/hive/ptest/execution/JIRAService.java 09c6d66
  testutils/ptest2/src/main/java/org/apache/hive/ptest/execution/JUnitReportParser.java 50cce25
  testutils/ptest2/src/main/java/org/apache/hive/ptest/execution/PTest.java f4f5076
  testutils/ptest2/src/main/java/org/apache/hive/ptest/execution/ReportingPhase.java 023c26b
  testutils/ptest2/src/main/java/org/apache/hive/ptest/execution/conf/QFileTestBatch.java 722312a
  testutils/ptest2/src/main/java/org/apache/hive/ptest/execution/conf/TestConfiguration.java fef2a3e
  testutils/ptest2/src/main/java/org/apache/hive/ptest/execution/conf/UnitTestBatch.java e911304
  testutils/ptest2/src/main/resources/batch-exec.vm a35fcec
  testutils/ptest2/src/main/resources/source-prep.vm 7a92272
  testutils/ptest2/src/test/java/org/apache/hive/ptest/execution/TestExecutionPhase.java eec5560
  testutils/ptest2/src/test/java/org/apache/hive/ptest/execution/TestHostExecutor.java 6fc649a
  testutils/ptest2/src/test/java/org/apache/hive/ptest/execution/TestReportingPhase.testExecute.approved.txt 19bc3e7
  testutils/ptest2/src/test/java/org/apache/hive/ptest/execution/TestScripts.java 0631996
  testutils/ptest2/src/test/java/org/apache/hive/ptest/execution/TestScripts.testBatch.approved.txt 9b4bb2f
  testutils/ptest2/src/test/java/org/apache/hive/ptest/execution/TestScripts.testPrepGit.approved.txt 2decd3e
  testutils/ptest2/src/test/java/org/apache/hive/ptest/execution/TestScripts.testPrepNone.approved.txt 7d8719d
  testutils/ptest2/src/test/java/org/apache/hive/ptest/execution/TestScripts.testPrepSvn.approved.txt 0af317e
  testutils/ptest2/src/test/resources/TEST-SomeTest-success.xml PRE-CREATION

Diff: https://reviews.apache.org/r/12790/diff/
Testing
-------

This code is currently running in our ptest environment.
Thanks,

Brock Noland