You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ambari.apache.org by Dmytro Sen <ds...@hortonworks.com> on 2015/03/18 22:24:15 UTC

Review Request 32218: BE: provide 'property_value_attributes' in stacks configuration API

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

Review request for Ambari, Myroslav Papirkovskyy and Srimanth Gunturi.


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


Repository: ambari


Description
-------

YARN properties should show up in the API as
{
  "href" : "http://c6401:8080/api/v1/stacks/HDP/versions/2.2/services/YARN/configurations/yarn.scheduler.minimum-allocation-mb",
  "StackConfigurations" : {
    "final" : "false",
    "property_description" : "The minimum allocation for every container request",
    "property_name" : "yarn.scheduler.minimum-allocation-mb",
    "property_type" : [ ],
    "property_value" : "512",// Recommended
    "property_value_attributes": {
        "type": "int",
        "minimum": "512",
        "maximum": "10240",
        "unit": "MB"
    },
    "property_depended_by": [
      {
        "property_type": "hive-site",
        "property_name": "hive.tez.container.size"
      },
      {
        "property_type": "mapred-site",
        
property_depends_on in the stack-definition should be transformed inversely to generate property_depended_by.


Diffs
-----

  ambari-server/src/main/java/org/apache/ambari/server/controller/StackConfigurationResponse.java 9a7831e38fcf400006e5323eb991696bbf3b4301 
  ambari-server/src/main/java/org/apache/ambari/server/controller/StackLevelConfigurationResponse.java 151ce07448353fdb6c6c6f0a0c37d3aacbb64f5d 
  ambari-server/src/main/java/org/apache/ambari/server/controller/internal/StackConfigurationResourceProvider.java 4abf3d28d74893f68829a799d01cb8a2b0fd6b43 
  ambari-server/src/main/java/org/apache/ambari/server/controller/internal/StackLevelConfigurationResourceProvider.java 1091bc7bac4076ac63eadb0afbd77eaef890122f 
  ambari-server/src/main/java/org/apache/ambari/server/stack/StackModule.java 4181293e40067ab95b2862f27e1c00f5d396f893 
  ambari-server/src/main/java/org/apache/ambari/server/state/PropertyDependencyInfo.java PRE-CREATION 
  ambari-server/src/main/java/org/apache/ambari/server/state/PropertyInfo.java e26b48f37559dfa7ca5ce1c24fe95b0142aed3f4 
  ambari-server/src/main/java/org/apache/ambari/server/state/ValueAttributesInfo.java PRE-CREATION 
  ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/configuration-mapred/mapred-site.xml 7955cb22cb3eb4677f743626a7eaf277c5881b48 
  ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/configuration/yarn-site.xml 3df629d7d0188bf8a1db63b7135b7be43aa1704f 
  ambari-server/src/main/resources/properties.json cdb2bd6546091383ea2c05313058d246d7160d8b 
  ambari-server/src/main/resources/stacks/HDP/2.2/services/YARN/configuration/yarn-site.xml 94b5304ba4026a7f324788788964b047a78a534b 
  ambari-server/src/test/java/org/apache/ambari/server/api/services/AmbariMetaInfoTest.java c7878db7e317c13ff92213863e18ad0390d69ad6 
  ambari-server/src/test/java/org/apache/ambari/server/state/PropertyInfoTest.java e8a2a8d71ca7fe249a00c9939898f5cf16fe67b0 
  ambari-server/src/test/resources/stacks/HDP/2.0.6/services/YARN/configuration/yarn-site.xml 59b5245183392103451e3d2f90e6e2feec64a0b2 

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


Testing
-------

all tests passed


Thanks,

Dmytro Sen


Re: Review Request 32218: BE: provide 'property_value_attributes' in stacks configuration API

Posted by Dmytro Sen <ds...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/32218/
-----------------------------------------------------------

(Updated Март 19, 2015, 2:57 п.п.)


Review request for Ambari, Myroslav Papirkovskyy and Srimanth Gunturi.


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


Repository: ambari


Description
-------

YARN properties should show up in the API as
{
  "href" : "http://c6401:8080/api/v1/stacks/HDP/versions/2.2/services/YARN/configurations/yarn.scheduler.minimum-allocation-mb",
  "StackConfigurations" : {
    "final" : "false",
    "property_description" : "The minimum allocation for every container request",
    "property_name" : "yarn.scheduler.minimum-allocation-mb",
    "property_type" : [ ],
    "property_value" : "512",// Recommended
    "property_value_attributes": {
        "type": "int",
        "minimum": "512",
        "maximum": "10240",
        "unit": "MB"
    },
    "property_depended_by": [
      {
        "property_type": "hive-site",
        "property_name": "hive.tez.container.size"
      },
      {
        "property_type": "mapred-site",
        
property_depends_on in the stack-definition should be transformed inversely to generate property_depended_by.


Diffs (updated)
-----

  ambari-server/src/main/java/org/apache/ambari/server/controller/StackConfigurationResponse.java 9a7831e 
  ambari-server/src/main/java/org/apache/ambari/server/controller/StackLevelConfigurationResponse.java 151ce07 
  ambari-server/src/main/java/org/apache/ambari/server/controller/internal/StackConfigurationResourceProvider.java 4abf3d2 
  ambari-server/src/main/java/org/apache/ambari/server/controller/internal/StackLevelConfigurationResourceProvider.java 1091bc7 
  ambari-server/src/main/java/org/apache/ambari/server/stack/StackModule.java 4181293 
  ambari-server/src/main/java/org/apache/ambari/server/state/PropertyDependencyInfo.java PRE-CREATION 
  ambari-server/src/main/java/org/apache/ambari/server/state/PropertyInfo.java e26b48f 
  ambari-server/src/main/java/org/apache/ambari/server/state/ValueAttributesInfo.java PRE-CREATION 
  ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/configuration-mapred/mapred-site.xml 7955cb2 
  ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/configuration/yarn-site.xml 3df629d 
  ambari-server/src/main/resources/properties.json cdb2bd6 
  ambari-server/src/main/resources/stacks/HDP/2.2/services/YARN/configuration/yarn-site.xml 94b5304 
  ambari-server/src/test/java/org/apache/ambari/server/api/services/AmbariMetaInfoTest.java c7878db 
  ambari-server/src/test/java/org/apache/ambari/server/state/PropertyInfoTest.java e8a2a8d 
  ambari-server/src/test/resources/stacks/HDP/2.0.6/services/YARN/configuration/yarn-site.xml 59b5245 

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


Testing
-------

all tests passed


Thanks,

Dmytro Sen


Re: Review Request 32218: BE: provide 'property_value_attributes' in stacks configuration API

Posted by Dmytro Sen <ds...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/32218/
-----------------------------------------------------------

(Updated Март 18, 2015, 9:46 п.п.)


Review request for Ambari, Myroslav Papirkovskyy and Srimanth Gunturi.


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


Repository: ambari


Description
-------

YARN properties should show up in the API as
{
  "href" : "http://c6401:8080/api/v1/stacks/HDP/versions/2.2/services/YARN/configurations/yarn.scheduler.minimum-allocation-mb",
  "StackConfigurations" : {
    "final" : "false",
    "property_description" : "The minimum allocation for every container request",
    "property_name" : "yarn.scheduler.minimum-allocation-mb",
    "property_type" : [ ],
    "property_value" : "512",// Recommended
    "property_value_attributes": {
        "type": "int",
        "minimum": "512",
        "maximum": "10240",
        "unit": "MB"
    },
    "property_depended_by": [
      {
        "property_type": "hive-site",
        "property_name": "hive.tez.container.size"
      },
      {
        "property_type": "mapred-site",
        
property_depends_on in the stack-definition should be transformed inversely to generate property_depended_by.


Diffs (updated)
-----

  ambari-server/src/main/java/org/apache/ambari/server/controller/StackConfigurationResponse.java 9a7831e 
  ambari-server/src/main/java/org/apache/ambari/server/controller/StackLevelConfigurationResponse.java 151ce07 
  ambari-server/src/main/java/org/apache/ambari/server/controller/internal/StackConfigurationResourceProvider.java 4abf3d2 
  ambari-server/src/main/java/org/apache/ambari/server/controller/internal/StackLevelConfigurationResourceProvider.java 1091bc7 
  ambari-server/src/main/java/org/apache/ambari/server/stack/StackModule.java 4181293 
  ambari-server/src/main/java/org/apache/ambari/server/state/PropertyDependencyInfo.java PRE-CREATION 
  ambari-server/src/main/java/org/apache/ambari/server/state/PropertyInfo.java e26b48f 
  ambari-server/src/main/java/org/apache/ambari/server/state/ValueAttributesInfo.java PRE-CREATION 
  ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/configuration-mapred/mapred-site.xml 7955cb2 
  ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/configuration/yarn-site.xml 3df629d 
  ambari-server/src/main/resources/properties.json cdb2bd6 
  ambari-server/src/main/resources/stacks/HDP/2.2/services/YARN/configuration/yarn-site.xml 94b5304 
  ambari-server/src/test/java/org/apache/ambari/server/api/services/AmbariMetaInfoTest.java c7878db 
  ambari-server/src/test/java/org/apache/ambari/server/state/PropertyInfoTest.java e8a2a8d 
  ambari-server/src/test/resources/stacks/HDP/2.0.6/services/YARN/configuration/yarn-site.xml 59b5245 

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


Testing
-------

all tests passed


Thanks,

Dmytro Sen