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

Review request for Ambari, Robert Nettleton and Sid Wagle.


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


Repository: ambari


Description
-------

Allow Hive and Atlas to be integrated by Ambari automatically when deploying 
via a blueprint.
- Introduce a new Atlas client component in the HDP stack which will be used 
for all Atlas service integration
- Set/Update the appropriate properties when Hive and Atlas are both included 
in the cluster topology

The end result of this work will be seamless integration of Hive and Atlas when 
deployed via a blueprint. The user will simply need to include the appropriate 
Hive and Atlas components and all configuration/integration is done 
automatically by Ambari.

This patch doesn't include any component depencency between HIVE and the ATLAS 
client so if the user doesn't specify an ATLAS client on the HIVE nodes, then 
they will need to manually configure the hive/atlas configurations.  This is 
because Ambari doesn't directly support conditional dependencies.  This should 
be added in another patch so that the ATLAS client is added as necessary to the 
blueprint during topology validation.


Diffs
-----

  
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/BlueprintConfigurationProcessor.java
 892cf32 
  ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/metainfo.xml 
a77bad3 
  
ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/package/scripts/atlas_client.py
 PRE-CREATION 
  
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/BlueprintConfigurationProcessorTest.java
 a97ca74 

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


Testing
-------

Functional Testing:
- Deployed clusters via blueprint with different topologies containing both 
Hive and Atlas.
- In the process of UI test to ensure that addition of ATLAS_CLIENT doesn't 
cause issues.

Unit Tests:
- Added new unit tests for BP config processor changes
- In the process of running full suite if unit tests

Will update the review with the above in-progress testing prior to merging.


Thanks,

John Speidel

Reply via email to