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

(Updated July 25, 2015, 8:45 a.m.)


Review request for Ambari, Jayush Luniya, Sid Wagle, and Yusaku Sako.


Bugs: AMBARI-12134
    https://issues.apache.org/jira/browse/AMBARI-12134


Repository: ambari


Description
-------

The ambari parent pom is targeted towards Java 1.6 while ambari uses 1.7 APIs. 
The ambari metrics pom is targeted at java 1.5 (by default) while 
ambari-metrics uses 1.6 APIs. This causes basic compilation failures, depending 
on how you build the projects and what tools you use to work on them. AFAIK 
Ambari 2.1 and beyond are targeted at 1.7, so the Java versions in the POMs 
should be updated to reflect this.

Aside from changing the target version for the compiler plugin, some other 
dependencies had to be updated to more recent versions that support 1.7 
compilation.


Diffs (updated)
-----

  ambari-metrics/ambari-metrics-flume-sink/pom.xml 01c2c7f 
  ambari-metrics/ambari-metrics-kafka-sink/pom.xml adb655b 
  ambari-metrics/ambari-metrics-storm-sink/pom.xml c4bc676 
  ambari-metrics/ambari-metrics-timelineservice/pom.xml 999005a 
  ambari-metrics/pom.xml b4f4ed3 
  ambari-project/pom.xml 0913bff 
  ambari-server/pom.xml 717d331 
  
ambari-server/src/test/java/org/apache/ambari/server/proxy/ProxyServiceTest.java
 ba8f01e 
  ambari-shell/ambari-groovy-shell/pom.xml 7d2f6da 
  contrib/views/hive/pom.xml 6a8a392 
  pom.xml ffb4d84 

Diff: https://reviews.apache.org/r/35844/diff/


Testing
-------

All tests pass!


File Attachments
----------------

AMBARI-12134-6.patch
  
https://reviews.apache.org/media/uploaded/files/2015/07/24/8526aa86-3aa4-4fa3-8316-91dec4a1adca__AMBARI-12134-6.patch


Thanks,

Jonathan Halterman

Reply via email to