-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63462/
-----------------------------------------------------------

(Updated Dec. 15, 2017, 4:46 p.m.)


Review request for atlas, Apoorv Naik, David Radley, Graham Wallis, Madhan 
Neethiraj, and Mandy Chessell.


Changes
-------

Changed message for java version too low to:
** JAVA VERSION ERROR ** Java 8 (Update 151) or above is required.

as per David's issue


Repository: atlas


Description
-------

ATLAS-2245: Update maven dependency to 3.5.0
Also updated Java Dependency to 8
Both are enforced in the top level POM using the maven enforcer plugin
Initial implementation used the 'dependencies' tag in the pom. However this is 
deprecated, not inherited, and was primarily intended for plugins rather than 
projects. 

THIS MEANS YOU CANNOT BUILD with earlier maven versions OR Java 7.
Many users will need to update maven in particular


Diffs (updated)
-----

  graphdb/janus/pom.xml 5fe38184 
  pom.xml 3b9a464b 


Diff: https://reviews.apache.org/r/63462/diff/6/

Changes: https://reviews.apache.org/r/63462/diff/5-6/


Testing
-------

Test with mvn clean install

If Maven version is not 3.5.0:
[INFO] --- maven-enforcer-plugin:3.0.0-M1:enforce (enforce-versions) @ 
apache-atlas ---
[WARNING] Rule 0: org.apache.maven.plugins.enforcer.RequireMavenVersion failed 
with message:
** Please upgrade maven to 3.5.2 or above. See 
https://maven.apache.org/install.html

Similar for Java

If dependencies met, build continues ok

Proposed (in Jira) to update wiki page & send out email on dist list due to 
developer impact
THIS MEANS YOU CANNOT BUILD with earlier maven versions OR Java 7.
Many users will need to update maven in particular

-- Update with 13 Dec patch --
* Tested with mvn clean install
* Checked JDK 1.8/151 or 152 works, that < 151 fails, and that Java 9 produces 
a warning
* Checked maven 3.5.0 or above works, earlier fails

Also checked build environment - from build logs build server is using maven 
3.5.2 & java 1.8.0_152

**** --> 
REVIEW QUESTION - I went with 3.5.0 (as Apache ranger hit bugs with 3.5.2.. 
though that was only with enuciate), and java 1.8.0_151 (as I know many have 
that, and it's easy to select when downloading). OK???
**** --<

Java HotSpot(TM) 64-Bit Server VM warning: ignoring option MaxPermSize=1024m; 
support was removed in 8.0
Apache Maven 3.5.2 (138edd61fd100ec658bfa2d307c43b76940a5d7d; 
2017-10-18T07:58:13Z)
Maven home: /home/jenkins/tools/maven/latest
Java version: 1.8.0_152, vendor: Oracle Corporation
Java home: /usr/local/asfpackages/java/jdk1.8.0_152/jre
Default locale: en_US, platform encoding: UTF-8
OS name: "linux", version: "4.4.0-43-generic", arch: "amd64", family: "unix"
+ java -version
java version "1.8.0_152"
Java(TM) SE Runtime Environment (build 1.8.0_152-b16)
Java HotSpot(TM) 64-Bit Server VM (build 25.152-b16, mixed mode)

Example of failure errors:

[INFO] --- maven-enforcer-plugin:3.0.0-M1:enforce (enforce-versions) @ 
apache-atlas ---
[WARNING] Rule 1: org.apache.maven.plugins.enforcer.RequireJavaVersion failed 
with message:
** JAVA VERSION ERROR ** Java 8 (b151) or above is required.


or

[INFO] --- maven-enforcer-plugin:3.0.0-M1:enforce (enforce-versions) @ 
apache-atlas ---
[WARNING] Rule 0: org.apache.maven.plugins.enforcer.RequireMavenVersion failed 
with message:
** MAVEN VERSION ERROR ** Maven 3.5.0 or above is required. See 
https://maven.apache.org/install.html
[WARNING] Rule 2: org.apache.maven.plugins.enforcer.RequireJavaVersion warned 
with message:
** JAVA VERSION WARNING ** Java 9 and above has not been tested with Atlas.


Thanks,

Nigel Jones

Reply via email to