You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Grant Henke (Code Review)" <ge...@cloudera.org> on 2018/04/23 03:35:39 UTC

[kudu-CR] WIP: Use Gradle for Jenkins testing

Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10150


Change subject: WIP: Use Gradle for Jenkins testing
......................................................................

WIP: Use Gradle for Jenkins testing

Change-Id: I4235d59187610469996439d6899da9ccbe61459d
---
M build-support/jenkins/build-and-test.sh
1 file changed, 19 insertions(+), 15 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/50/10150/1
-- 
To view, visit http://gerrit.cloudera.org:8080/10150
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I4235d59187610469996439d6899da9ccbe61459d
Gerrit-Change-Number: 10150
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke <gr...@apache.org>

[kudu-CR] WIP: Use Gradle for Jenkins testing

Posted by "Grant Henke (Code Review)" <ge...@cloudera.org>.
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/10150 )

Change subject: WIP: Use Gradle for Jenkins testing
......................................................................


Patch Set 2:

That sounds like a good plan.


-- 
To view, visit http://gerrit.cloudera.org:8080/10150
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I4235d59187610469996439d6899da9ccbe61459d
Gerrit-Change-Number: 10150
Gerrit-PatchSet: 2
Gerrit-Owner: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Comment-Date: Mon, 23 Apr 2018 03:48:20 +0000
Gerrit-HasComments: No

[kudu-CR] WIP: Use Gradle for Jenkins testing

Posted by "Grant Henke (Code Review)" <ge...@cloudera.org>.
Grant Henke has abandoned this change. ( http://gerrit.cloudera.org:8080/10150 )

Change subject: WIP: Use Gradle for Jenkins testing
......................................................................


Abandoned
-- 
To view, visit http://gerrit.cloudera.org:8080/10150
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: abandon
Gerrit-Change-Id: I4235d59187610469996439d6899da9ccbe61459d
Gerrit-Change-Number: 10150
Gerrit-PatchSet: 2
Gerrit-Owner: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Kudu Jenkins

[kudu-CR] WIP: Use Gradle for Jenkins testing

Posted by "Adar Dembo (Code Review)" <ge...@cloudera.org>.
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/10150 )

Change subject: WIP: Use Gradle for Jenkins testing
......................................................................


Patch Set 2:

I'm a fan, but I'd rather see it done a little bit differently:
1. A patch to make the Gradle and Maven builds more symmetric in build-and-test.sh and exposes a BUILD_FOO env variable for each.
2. A patch that toggles the policy by changing the default value for those env variables.


-- 
To view, visit http://gerrit.cloudera.org:8080/10150
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I4235d59187610469996439d6899da9ccbe61459d
Gerrit-Change-Number: 10150
Gerrit-PatchSet: 2
Gerrit-Owner: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Comment-Date: Mon, 23 Apr 2018 03:46:30 +0000
Gerrit-HasComments: No

[kudu-CR] WIP: Use Gradle for Jenkins testing

Posted by "Grant Henke (Code Review)" <ge...@cloudera.org>.
Hello Kudu Jenkins, 

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/10150

to look at the new patch set (#2).

Change subject: WIP: Use Gradle for Jenkins testing
......................................................................

WIP: Use Gradle for Jenkins testing

Change-Id: I4235d59187610469996439d6899da9ccbe61459d
---
M build-support/jenkins/build-and-test.sh
1 file changed, 13 insertions(+), 15 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/50/10150/2
-- 
To view, visit http://gerrit.cloudera.org:8080/10150
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I4235d59187610469996439d6899da9ccbe61459d
Gerrit-Change-Number: 10150
Gerrit-PatchSet: 2
Gerrit-Owner: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Kudu Jenkins