----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59629/ -----------------------------------------------------------
Review request for hive, Sergio Pena and Siddharth Seth. Bugs: HIVE-14745 https://issues.apache.org/jira/browse/HIVE-14745 Repository: hive-git Description ------- Added the ability to write data needed for the jira comment into json. Refactored the TestStatus response to contain the generated jira comment. Refactored the PTestClient to take the jira comment from the last TestStatus response which is either a success or a failure and post it to the Jira Service. Diffs ----- testutils/ptest2/conf/deployed/master-mr2.properties 05c040599172caf9d67c4e27affce12a335dab4b testutils/ptest2/src/main/java/org/apache/hive/ptest/api/client/PTestClient.java 8e2604d372ac29b94445b269f08423b058308efe testutils/ptest2/src/main/java/org/apache/hive/ptest/api/common/JiraCommentPoster.java PRE-CREATION testutils/ptest2/src/main/java/org/apache/hive/ptest/api/response/TestStatus.java 7dfadb42edbb8526b169d3034e01d0e8e8739fdf testutils/ptest2/src/main/java/org/apache/hive/ptest/api/server/ExecutionController.java 2f96ad03023e9f51d44d203f34edd04183605a22 testutils/ptest2/src/main/java/org/apache/hive/ptest/api/server/Test.java 7fbabd96d8c9a443bb5cfd2defe7a10510cf9a15 testutils/ptest2/src/main/java/org/apache/hive/ptest/api/server/TestExecutor.java b2c61f03c5bf5f170894141848c89fc26129115a testutils/ptest2/src/main/java/org/apache/hive/ptest/execution/JIRAService.java ff1dbd4bafaba9a0e88321c6507a47bbe0421833 testutils/ptest2/src/main/java/org/apache/hive/ptest/execution/PTest.java 1cdfdb309acd8282e593abd7ed10c87721926c60 testutils/ptest2/src/main/java/org/apache/hive/ptest/execution/conf/TestConfiguration.java e584f9c105fa134e3e267d6c6817d441b4c6b249 testutils/ptest2/src/test/java/org/apache/hive/ptest/api/client/TestPTestClient.java PRE-CREATION testutils/ptest2/src/test/java/org/apache/hive/ptest/api/server/TestTestExecutor.java a4a789b579305d9ed573d8c1fd0b6ce75787d50f testutils/ptest2/src/test/java/org/apache/hive/ptest/execution/TestJIRAService.java b97b890dfe855539de2696788327ba9b4a841ff3 Diff: https://reviews.apache.org/r/59629/diff/1/ Testing ------- Added unit tests for PTestClient and JiraService. Ran a test with a heavily dumbed down CloudExecutionContextProvider that would create the drowns on the local machine isntead of a cloud povider to test integration between PTestClient and the WebService. Thanks, Barna Zsombor Klara