-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/38838/
-----------------------------------------------------------
Review request for hive, Brock Noland and Sergio Pena.
Bugs: HIVE-11913
https://issues.apache.org/jira/browse/HIVE-11913
Repository: hive-git
Description
-------
Make HiveQA give -1 if there are no tests added or modified (Test.*.java or
.*.q) in the git patch. Basic idea is to add a new test phase "TestCheckPhase"
to read the git patch and parse for newly added tests, and channel this
information to the JIRA reporting service.
There are some changes as I also did auto-cleanup the JIRAService class to meet
the coding guidelines.
Diffs
-----
testutils/ptest2/src/main/java/org/apache/hive/ptest/execution/JIRAService.java
c7be572
testutils/ptest2/src/main/java/org/apache/hive/ptest/execution/PTest.java
7217ef9
testutils/ptest2/src/main/java/org/apache/hive/ptest/execution/TestCheckPhase.java
PRE-CREATION
testutils/ptest2/src/test/java/org/apache/hive/ptest/execution/TestTestCheckPhase.java
PRE-CREATION
testutils/ptest2/src/test/resources/HIVE-10761.6.patch PRE-CREATION
testutils/ptest2/src/test/resources/HIVE-11271.4.patch PRE-CREATION
testutils/ptest2/src/test/resources/HIVE-9377.1.patch PRE-CREATION
testutils/ptest2/src/test/resources/remove-test.patch PRE-CREATION
Diff: https://reviews.apache.org/r/38838/diff/
Testing
-------
Added unit tests. Will need to see in production.
Thanks,
Szehon Ho
