Adar Dembo has posted comments on this change.

Change subject: Bump minimum maven version to 3.1.0
......................................................................


Patch Set 1:

(1 comment)

> hrm, given I feel like most people use a non-OS-provided 'mvn',
 > maybe it's not too big a deal to bump the minimum version? We can
 > get our precommit boxes to have a newer mvn installed via tarball,
 > etc.

+1

http://gerrit.cloudera.org:8080/#/c/7105/1/java/pom.xml
File java/pom.xml:

Line 108:                                 <version>[3.1.0,)</version>
Please add a comment explaining why we need 3.1.0.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ie397a7b9bee8e2d30e70934b40e8fd190f318e4d
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Dan Burkert <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Dan Burkert <[email protected]>
Gerrit-Reviewer: Jean-Daniel Cryans <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <[email protected]>
Gerrit-Reviewer: Will Berkeley <[email protected]>
Gerrit-HasComments: Yes

Reply via email to