You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hive.apache.org by br...@apache.org on 2014/10/17 18:50:47 UTC
svn commit: r1632632 -
/hive/trunk/testutils/ptest2/src/main/java/org/apache/hive/ptest/execution/JIRAService.java
Author: brock
Date: Fri Oct 17 16:50:47 2014
New Revision: 1632632
URL: http://svn.apache.org/r1632632
Log:
HIVE-8491 - Fix build name in ptest pre-commit message (Mohit Sabharwal via Brock)
Modified:
hive/trunk/testutils/ptest2/src/main/java/org/apache/hive/ptest/execution/JIRAService.java
Modified: hive/trunk/testutils/ptest2/src/main/java/org/apache/hive/ptest/execution/JIRAService.java
URL: http://svn.apache.org/viewvc/hive/trunk/testutils/ptest2/src/main/java/org/apache/hive/ptest/execution/JIRAService.java?rev=1632632&r1=1632631&r2=1632632&view=diff
==============================================================================
--- hive/trunk/testutils/ptest2/src/main/java/org/apache/hive/ptest/execution/JIRAService.java (original)
+++ hive/trunk/testutils/ptest2/src/main/java/org/apache/hive/ptest/execution/JIRAService.java Fri Oct 17 16:50:47 2014
@@ -135,13 +135,9 @@ class JIRAService {
}
comments.add("This message is automatically generated.");
String attachmentId = parseAttachementId(mPatch);
- if(!attachmentId.isEmpty()) {
- comments.add("");
- comments.add("ATTACHMENT ID: " + attachmentId);
- }
- if(!buildInfo.getBuildName().isEmpty()) {
- comments.add(" - " + buildInfo.getBuildName());
- }
+ comments.add("");
+ comments.add("ATTACHMENT ID: " + attachmentId +
+ " - " + buildInfo.getBuildName());
mLogger.info("Comment: " + Joiner.on("\n").join(comments));
String body = Joiner.on("\n").join(comments);
String url = String.format("%s/rest/api/2/issue/%s/comment", mUrl, mName);