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

Review request for atlas, Ashutosh Mestry, Madhan Neethiraj, and Sarath 
Subramanian.


Bugs: ATLAS-3966
    https://issues.apache.org/jira/browse/ATLAS-3966


Repository: atlas


Description
-------

If a property is defined multiple times with different values in the 
application configuration file, while loading the proper, the oldest value is 
read. Technically this is correct behavior. However many times, instead of 
updating the value of a particular property, its convenient to just append the 
property again with a new value in the configuration file.

This ask is specifically for a property called "atlas.metadata.namespace"

AtlasConfigurationUtil Class has been created with static method 
getLatesttString.
All the places where atlas.metadata.namespace property is read from 
configuration is now read through the new Util method.
Unit test for the new method is also added.


Diffs
-----

  
addons/hbase-bridge/src/main/java/org/apache/atlas/hbase/bridge/HBaseBridge.java
 f01266445 
  
addons/hive-bridge/src/main/java/org/apache/atlas/hive/bridge/HiveMetaStoreBridge.java
 24e06dc2b 
  
addons/kafka-bridge/src/main/java/org/apache/atlas/kafka/bridge/KafkaBridge.java
 40b1fee71 
  addons/sqoop-bridge/src/main/java/org/apache/atlas/sqoop/hook/SqoopHook.java 
25ab7cb3c 
  intg/src/main/java/org/apache/atlas/utils/AtlasConfigurationUtil.java 
PRE-CREATION 
  intg/src/test/java/org/apache/atlas/ApplicationPropertiesTest.java a002a7a8a 
  intg/src/test/resources/test.properties 395537f30 
  notification/src/main/java/org/apache/atlas/hook/AtlasHook.java 8659126eb 


Diff: https://reviews.apache.org/r/72921/diff/1/


Testing
-------

Local Build and Testing done. PreCommit Build Test is in progress.


Thanks,

Deep Singh

Reply via email to