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/10/12 15:55:14 UTC
[kudu-CR] WIP: Remove the Maven Build
Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/11667
Change subject: WIP: Remove the Maven Build
......................................................................
WIP: Remove the Maven Build
This patch removes the Maven poms and remaining
references to the Maven build. Going forward the
Gradle build will be used.
Change-Id: If3c1b6b5bc2218a6285bde10d9ab7e1d9013a023
---
M RELEASING.adoc
M build-support/jenkins/build-and-test.sh
M build-support/jenkins/post-build-clean.sh
M docs/developing.adoc
M docs/installation.adoc
M docs/support/scripts/make_site.sh
M java/README.adoc
D java/kudu-backup/pom.xml
D java/kudu-client-tools/pom.xml
D java/kudu-client/pom.xml
D java/kudu-flume-sink/pom.xml
D java/kudu-hive/pom.xml
M java/kudu-jepsen/README.adoc
D java/kudu-jepsen/pom.xml
D java/kudu-mapreduce/pom.xml
D java/kudu-spark-tools/pom.xml
D java/kudu-spark/pom.xml
D java/pom.xml
18 files changed, 60 insertions(+), 2,222 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/67/11667/1
--
To view, visit http://gerrit.cloudera.org:8080/11667
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: If3c1b6b5bc2218a6285bde10d9ab7e1d9013a023
Gerrit-Change-Number: 11667
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke <gr...@apache.org>
[kudu-CR] [Java] Remove the Maven Build
Posted by "Grant Henke (Code Review)" <ge...@cloudera.org>.
Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11667 )
Change subject: [Java] Remove the Maven Build
......................................................................
[Java] Remove the Maven Build
This patch removes the Maven poms and remaining
references to the Maven build. Going forward the
Gradle build will be used.
Change-Id: If3c1b6b5bc2218a6285bde10d9ab7e1d9013a023
Reviewed-on: http://gerrit.cloudera.org:8080/11667
Reviewed-by: Dan Burkert <da...@apache.org>
Tested-by: Kudu Jenkins
---
M RELEASING.adoc
M build-support/jenkins/build-and-test.sh
M build-support/jenkins/post-build-clean.sh
M docs/developing.adoc
M docs/installation.adoc
M docs/support/scripts/make_site.sh
M java/README.adoc
D java/kudu-backup/pom.xml
D java/kudu-client-tools/pom.xml
D java/kudu-client/pom.xml
D java/kudu-flume-sink/pom.xml
D java/kudu-hive/pom.xml
M java/kudu-jepsen/README.adoc
D java/kudu-jepsen/pom.xml
D java/kudu-mapreduce/pom.xml
D java/kudu-spark-tools/pom.xml
D java/kudu-spark/pom.xml
D java/pom.xml
18 files changed, 60 insertions(+), 2,222 deletions(-)
Approvals:
Dan Burkert: Looks good to me, approved
Kudu Jenkins: Verified
--
To view, visit http://gerrit.cloudera.org:8080/11667
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: If3c1b6b5bc2218a6285bde10d9ab7e1d9013a023
Gerrit-Change-Number: 11667
Gerrit-PatchSet: 4
Gerrit-Owner: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Dan Burkert <da...@apache.org>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
[kudu-CR] WIP: Remove the Maven Build
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/11667
to look at the new patch set (#2).
Change subject: WIP: Remove the Maven Build
......................................................................
WIP: Remove the Maven Build
This patch removes the Maven poms and remaining
references to the Maven build. Going forward the
Gradle build will be used.
Change-Id: If3c1b6b5bc2218a6285bde10d9ab7e1d9013a023
---
M RELEASING.adoc
M build-support/jenkins/build-and-test.sh
M build-support/jenkins/post-build-clean.sh
M docs/developing.adoc
M docs/installation.adoc
M docs/support/scripts/make_site.sh
M java/README.adoc
D java/kudu-backup/pom.xml
D java/kudu-client-tools/pom.xml
D java/kudu-client/pom.xml
D java/kudu-flume-sink/pom.xml
D java/kudu-hive/pom.xml
M java/kudu-jepsen/README.adoc
D java/kudu-jepsen/pom.xml
D java/kudu-mapreduce/pom.xml
D java/kudu-spark-tools/pom.xml
D java/kudu-spark/pom.xml
D java/pom.xml
18 files changed, 60 insertions(+), 2,222 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/67/11667/2
--
To view, visit http://gerrit.cloudera.org:8080/11667
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: If3c1b6b5bc2218a6285bde10d9ab7e1d9013a023
Gerrit-Change-Number: 11667
Gerrit-PatchSet: 2
Gerrit-Owner: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
[kudu-CR] [Java] Remove the Maven Build
Posted by "Dan Burkert (Code Review)" <ge...@cloudera.org>.
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/11667 )
Change subject: [Java] Remove the Maven Build
......................................................................
Patch Set 3: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/11667
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: If3c1b6b5bc2218a6285bde10d9ab7e1d9013a023
Gerrit-Change-Number: 11667
Gerrit-PatchSet: 3
Gerrit-Owner: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Dan Burkert <da...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Comment-Date: Tue, 16 Oct 2018 18:14:49 +0000
Gerrit-HasComments: No
[kudu-CR] [Java] Remove the Maven Build
Posted by "Grant Henke (Code Review)" <ge...@cloudera.org>.
Hello Dan Burkert, Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/11667
to look at the new patch set (#3).
Change subject: [Java] Remove the Maven Build
......................................................................
[Java] Remove the Maven Build
This patch removes the Maven poms and remaining
references to the Maven build. Going forward the
Gradle build will be used.
Change-Id: If3c1b6b5bc2218a6285bde10d9ab7e1d9013a023
---
M RELEASING.adoc
M build-support/jenkins/build-and-test.sh
M build-support/jenkins/post-build-clean.sh
M docs/developing.adoc
M docs/installation.adoc
M docs/support/scripts/make_site.sh
M java/README.adoc
D java/kudu-backup/pom.xml
D java/kudu-client-tools/pom.xml
D java/kudu-client/pom.xml
D java/kudu-flume-sink/pom.xml
D java/kudu-hive/pom.xml
M java/kudu-jepsen/README.adoc
D java/kudu-jepsen/pom.xml
D java/kudu-mapreduce/pom.xml
D java/kudu-spark-tools/pom.xml
D java/kudu-spark/pom.xml
D java/pom.xml
18 files changed, 60 insertions(+), 2,222 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/67/11667/3
--
To view, visit http://gerrit.cloudera.org:8080/11667
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: If3c1b6b5bc2218a6285bde10d9ab7e1d9013a023
Gerrit-Change-Number: 11667
Gerrit-PatchSet: 3
Gerrit-Owner: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Dan Burkert <da...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
[kudu-CR] WIP: Remove the Maven Build
Posted by "Dan Burkert (Code Review)" <ge...@cloudera.org>.
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/11667 )
Change subject: WIP: Remove the Maven Build
......................................................................
Patch Set 2:
LGTM
--
To view, visit http://gerrit.cloudera.org:8080/11667
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: If3c1b6b5bc2218a6285bde10d9ab7e1d9013a023
Gerrit-Change-Number: 11667
Gerrit-PatchSet: 2
Gerrit-Owner: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Dan Burkert <da...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Comment-Date: Mon, 15 Oct 2018 21:18:29 +0000
Gerrit-HasComments: No