You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by bu...@apache.org on 2017/09/14 04:00:01 UTC
[40/50] [abbrv] hbase git commit: HBASE-18467 remove for-each loops
since that does CPS which breaks.
HBASE-18467 remove for-each loops since that does CPS which breaks.
Project: http://git-wip-us.apache.org/repos/asf/hbase/repo
Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/2ae43046
Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/2ae43046
Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/2ae43046
Branch: refs/heads/HBASE-18467
Commit: 2ae4304605c15c42806ab8e60e88b644336b5951
Parents: 5c50b3b
Author: Sean Busbey <bu...@apache.org>
Authored: Sun Aug 27 10:46:51 2017 -0500
Committer: Sean Busbey <bu...@apache.org>
Committed: Wed Sep 13 22:51:11 2017 -0500
----------------------------------------------------------------------
dev-support/Jenkinsfile | 10 +++++++---
1 file changed, 7 insertions(+), 3 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/hbase/blob/2ae43046/dev-support/Jenkinsfile
----------------------------------------------------------------------
diff --git a/dev-support/Jenkinsfile b/dev-support/Jenkinsfile
index 50142bf..5b6bb9e 100644
--- a/dev-support/Jenkinsfile
+++ b/dev-support/Jenkinsfile
@@ -394,13 +394,16 @@ END
echo ""
echo "[INFO] There are ${currentBuild.changeSets.size()} change sets."
def seenJiras = []
- for ( changelist in currentBuild.changeSets ) {
+ for (int i = 0; i < currentBuild.changeSets.size(); i++) {
+ def changelist = currentBuild.changeSets.get(i)
if ( changelist.isEmptySet() ) {
echo "[DEBUG] change set was empty, skipping JIRA comments."
} else {
echo "[DEBUG] there are changes in the change set. Attempting to post comments."
}
- for ( change in changelist ) {
+ def changes = changelist.getLogs()
+ for (int j = 0; j < changes.size(); j++) {
+ def change = changes.get(j)
CharSequence msg = change.msg
echo "[DEBUG] msg is of class ${msg.class}"
echo "change: ${change}"
@@ -417,7 +420,8 @@ END
if (jiras.length == 0) {
echo "[WARN] no JIRA key found in message, TODO email committer"
}
- for (currentIssue in jiras) {
+ for (int k = 0; k < jiras.length; k++) {
+ def currentIssue = jiras[k]
echo "[DEBUG] found jira key: ${currentIssue}"
if ( currentIssue in seenJiras ) {
echo "[DEBUG] already commented on ${currentIssue}."