ambari-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Andrew Onischuk" <aonis...@hortonworks.com>
Subject Re: Review Request 24886: Stacks API should expose service level dependencies
Date Wed, 20 Aug 2014 12:20:45 GMT

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

(Updated Aug. 20, 2014, 12:20 p.m.)


Review request for Ambari, Dmitro Lisnichenko and Vitalyi Brodetskyi.


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


Repository: ambari


Description
-------

Currently stack API exposes component level dependencies which does not roll
up to service level dependency is required for restricting user to install a
service that cannot work in the absence of a dependent service.

Currently there is a hard-coded map in the ambari-web code for service
dependency which restricts this validation functionality to only known
services of HDP stack.  
service dependency map:

    
    
    
    HDP-1: {
        HDFS: ['MAPREDUCE', ''HBASE', SQOOP'],
        MAPREDUCE: ['PIG', 'OOZIE', 'HIVE'],
        ZOOKEEPER: ['HBASE', 'HIVE', 'WEBHCAT'],
        HIVE: ['HCATALOG','WEBHCAT']
      },
      HDP-2: {
        ZOOKEEPER: ['HDFS', 'HBASE', 'HIVE', 'WEBHCAT', 'STORM']
        HDFS: ['YARN', 'HBASE', 'FLUME', 'SQOOP'],
        YARN: [ 'MAPREDUCE2', 'PIG', 'OOZIE', 'HIVE', 'TEZ'],
        TEZ: ['YARN', 'HIVE'],
        OOZIE: ['FALCON'], 
        HIVE: ['HCATALOG','WEBHCAT'] 
      }
    


Diffs (updated)
-----

  ambari-server/src/main/java/org/apache/ambari/server/api/util/StackExtensionHelper.java
0670d9c 
  ambari-server/src/main/java/org/apache/ambari/server/controller/StackServiceResponse.java
25b3b8d 
  ambari-server/src/main/java/org/apache/ambari/server/controller/internal/StackServiceResourceProvider.java
0523edc 
  ambari-server/src/main/java/org/apache/ambari/server/state/ServiceInfo.java ac1c9b5 
  ambari-server/src/main/resources/properties.json aa2bf71 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HBASE/metainfo.xml d1e8795 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HIVE/metainfo.xml c1263d5 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/MAPREDUCE/metainfo.xml a900f2e

  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/OOZIE/metainfo.xml 4e83703 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/PIG/metainfo.xml e7f0cad 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/SQOOP/metainfo.xml 1f835cb 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/WEBHCAT/metainfo.xml 636f499

  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/FLUME/metainfo.xml 4011c28 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HBASE/metainfo.xml c63b2b8 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HDFS/metainfo.xml 62b7cd8 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HIVE/metainfo.xml eecc292 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/OOZIE/metainfo.xml bdc83ea 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/PIG/metainfo.xml 48b42f1 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/SQOOP/metainfo.xml 4573211 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/WEBHCAT/metainfo.xml 325fddf

  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/YARN/metainfo.xml 8bb8dd7 
  ambari-server/src/main/resources/stacks/HDP/2.1/services/FALCON/metainfo.xml 43c2a3e 
  ambari-server/src/main/resources/stacks/HDP/2.1/services/STORM/metainfo.xml 3f13983 
  ambari-server/src/main/resources/stacks/HDP/2.1/services/TEZ/metainfo.xml e6ffd18 
  ambari-server/src/test/java/org/apache/ambari/server/api/util/StackExtensionHelperTest.java
517f3b7 
  ambari-server/src/test/resources/stacks/HDP/2.0.7/services/HBASE/metainfo.xml c114c83 

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


Testing
-------

mvn clean test


Thanks,

Andrew Onischuk


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message