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

(Updated Sept. 8, 2015, 3:17 a.m.)


Review request for oodt.


Changes
-------

Chris please see the new patch. I slackened up some type checking across a 
couple of modules which I introduced in the first effort. Incorrectly typed 
statements were the cause of the API breaks. These are all now resolved.


Bugs: OODT-872
    https://issues.apache.org/jira/browse/OODT-872


Repository: oodt


Description
-------

We have been having some problems recently with older Jena versioning.
I decided to just upgrade our use of Jena within profile server and grid 
components.


Diffs (updated)
-----

  trunk/core/pom.xml 1701716 
  trunk/grid/pom.xml 1701716 
  
trunk/opendapps/src/main/java/org/apache/oodt/opendapps/OpendapProfileElementExtractor.java
 1701716 
  trunk/profile/pom.xml 1701716 
  
trunk/profile/src/main/java/org/apache/oodt/profile/EnumeratedProfileElement.java
 1701716 
  trunk/profile/src/main/java/org/apache/oodt/profile/Profile.java 1701716 
  trunk/profile/src/main/java/org/apache/oodt/profile/ProfileElement.java 
1701716 
  trunk/profile/src/main/java/org/apache/oodt/profile/RangedProfileElement.java 
1701716 
  trunk/profile/src/main/java/org/apache/oodt/profile/ResourceAttributes.java 
1701716 
  
trunk/profile/src/main/java/org/apache/oodt/profile/UnspecifiedProfileElement.java
 1701716 
  trunk/profile/src/main/java/org/apache/oodt/profile/Utility.java 1701716 
  trunk/xmlps/src/main/java/org/apache/oodt/xmlps/profile/DBMSExecutor.java 
1701716 

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


Testing
-------

Tests on profile server and grid componenets pass. 
Had to remove the <additionalparam>-Xdoclint:none</additionalparam> from 
core/pom.xml due to javac 8 restrictions. 
Also stepped through a good bit of code and made implemented generic type 
safety.


Thanks,

Lewis McGibbney

Reply via email to