You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@hive.apache.org by Mohit Sabharwal <mo...@cloudera.com> on 2014/10/15 02:43:42 UTC

Review Request 26717: HIVE-8351 : Add build name to ptest pre-commit message

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

Review request for hive.


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


Repository: hive-git


Description
-------

HIVE-8351 : Add build name to ptest pre-commit message

Currently, the message is uniquely identified by the attachment-id,
and we only choose to test the one with the latest (highest) attachment-id.

This patch adds the build name to the pre-commit message, so we can still
distinguish between hadoop-2 and hadoop-1 (and run the test twice, one for
each profile).


Diffs
-----

  testutils/ptest2/src/main/java/org/apache/hive/ptest/execution/JIRAService.java ca59ac48d7401989948989e3fa8466ce00688b1f 
  testutils/ptest2/src/main/java/org/apache/hive/ptest/execution/PTest.java 3455137f9a9a1a17acd72ffc4b4c90b7ac50593e 
  testutils/ptest2/src/test/java/org/apache/hive/ptest/execution/TestJIRAService.java 121d7ac8e4e84ddf285ce055b9e8c01e83fccb9e 

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


Testing
-------

Enhanced unit test to verify build name is correctly extracted.


Thanks,

Mohit Sabharwal