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


Ship it!




Ship It!

- Hemanth Yamijala


On June 21, 2016, 6:20 a.m., Madhan Neethiraj wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/49002/
> -----------------------------------------------------------
> 
> (Updated June 21, 2016, 6:20 a.m.)
> 
> 
> Review request for atlas.
> 
> 
> Bugs: ATLAS-922
>     https://issues.apache.org/jira/browse/ATLAS-922
> 
> 
> Repository: atlas
> 
> 
> Description
> -------
> 
> remove embedded atlas-application.properties
> 
> 
> Diffs
> -----
> 
>   addons/falcon-bridge/pom.xml 14c6090 
>   addons/hdfs-model/pom.xml f4033bc 
>   addons/hive-bridge/pom.xml dff93a9 
>   addons/sqoop-bridge/pom.xml 343bb4e 
>   addons/storm-bridge/pom.xml b1a7a9b 
>   catalog/pom.xml 8a49d3d 
>   client/pom.xml 69b8731 
>   common/src/main/java/org/apache/atlas/ApplicationProperties.java 48cb7a6 
>   notification/pom.xml fc08115 
>   pom.xml 1539922 
>   repository/pom.xml 533e48b 
>   typesystem/pom.xml c1f3fcd 
>   typesystem/src/main/resources/atlas-application.properties 2bbcdcd 
>   typesystem/src/test/resources/atlas-application.properties ba66ae5 
>   webapp/pom.xml e17f64d 
> 
> Diff: https://reviews.apache.org/r/49002/diff/
> 
> 
> Testing
> -------
> 
> Verified that atlas-application.properties is not embedded in 
> atlas-typesystem.jar and the application/hooks load 
> atlas-application.properties from application's conf directory (in the 
> CLASSPATH) - irrespective of where atlas jars appear in the CLASSPATH.
> 
> 
> Thanks,
> 
> Madhan Neethiraj
> 
>

Reply via email to