Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/7105
to look at the new patch set (#2).
Change subject: Bump minimum maven version to 3.1.0
......................................................................
Bump minimum maven version to 3.1.0
We're now relying on the os-maven-plugin in order to locate the proper
protoc binary. os-maven-plugin has a minimum version dependency of
3.1.0. Because we're not meeting this requirement, we get the following
warning in our builds:
[WARNING] Failed to inject repository session properties.
java.lang.NoSuchMethodError:
org.apache.maven.execution.MavenSession.getRepositorySession()Lorg/eclipse/aether/RepositorySystemSession;
at
kr.motd.maven.os.RepositorySessionInjector.injectRepositorySession(RepositorySessionInjector.java:22)
at
kr.motd.maven.os.DetectExtension.injectSession(DetectExtension.java:151)
at
kr.motd.maven.os.DetectExtension.afterProjectsRead(DetectExtension.java:110)
at org.apache.maven.DefaultMaven.doExecute(DefaultMaven.java:274)
at org.apache.maven.DefaultMaven.execute(DefaultMaven.java:156)
at org.apache.maven.cli.MavenCli.execute(MavenCli.java:537)
at org.apache.maven.cli.MavenCli.doMain(MavenCli.java:196)
at org.apache.maven.cli.MavenCli.main(MavenCli.java:141)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at
sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
at
sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:498)
at
org.codehaus.plexus.classworlds.launcher.Launcher.launchEnhanced(Launcher.java:290)
at
org.codehaus.plexus.classworlds.launcher.Launcher.launch(Launcher.java:230)
at
org.codehaus.plexus.classworlds.launcher.Launcher.mainWithExitCode(Launcher.java:414)
at
org.codehaus.plexus.classworlds.launcher.Launcher.main(Launcher.java:357)
This is tracked upstream in trustin/os-maven-plugin/issues/21.
The warning seems relatively innocuous for us, we haven't seen any
issues yet. There is a real downside to merging this patch - Centos 7
comes with Maven 3.0.0 out of the box.
Change-Id: Ie397a7b9bee8e2d30e70934b40e8fd190f318e4d
---
M java/pom.xml
1 file changed, 2 insertions(+), 1 deletion(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/05/7105/2
--
To view, visit http://gerrit.cloudera.org:8080/7105
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ie397a7b9bee8e2d30e70934b40e8fd190f318e4d
Gerrit-PatchSet: 2
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]>
