You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ambari.apache.org by Andrew Onischuk <ao...@hortonworks.com> on 2014/09/02 19:01:38 UTC

Review Request 25256: Stack Service Pluggability: Adding service to a stack requires editing hooks folder

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

Review request for Ambari and Dmitro Lisnichenko.


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


Repository: ambari


Description
-------

Following files has service specific references.

  1. hooks->BEFORE-INSTALL->shared_initialization.py
  2. hooks->BEFORE-START->shared_initialization.py

hooks->BEFORE-INSTALL->shared_initialization.py file is required to change
while adding a new service package because user creation logic for a service
resides in the hooks folder.

Any tasks that needs to be done before installing or starting a service should
be implemented in a service package.

**On accomplishing this ticket, It will be possible to plug a new service definition into a deployed cluster without changing any existing python code.**


Diffs
-----

  ambari-server/src/main/java/org/apache/ambari/server/agent/ExecutionCommand.java 33c5af4 
  ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementControllerImpl.java efce9bd 
  ambari-server/src/main/java/org/apache/ambari/server/controller/StackConfigurationResponse.java 96ca232 
  ambari-server/src/main/java/org/apache/ambari/server/controller/StackLevelConfigurationResponse.java 7229197 
  ambari-server/src/main/java/org/apache/ambari/server/controller/internal/BlueprintResourceProvider.java 2cf01ff 
  ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ClientConfigResourceProvider.java e5b6016 
  ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ClusterResourceProvider.java eeee694 
  ambari-server/src/main/java/org/apache/ambari/server/controller/internal/StackConfigurationResourceProvider.java 2e2bfe1 
  ambari-server/src/main/java/org/apache/ambari/server/controller/internal/StackLevelConfigurationResourceProvider.java 116f589 
  ambari-server/src/main/java/org/apache/ambari/server/orm/entities/BlueprintEntity.java 7926fd4 
  ambari-server/src/main/java/org/apache/ambari/server/state/ConfigHelper.java 43838d7 
  ambari-server/src/main/java/org/apache/ambari/server/state/PropertyInfo.java 6a17e9d 
  ambari-server/src/main/resources/properties.json 741ffe6 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/configuration/cluster-env.xml a2df35a 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/before-INSTALL/scripts/params.py 2bcbd50 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/before-INSTALL/scripts/shared_initialization.py f556335 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/GANGLIA/configuration/ganglia-env.xml 68d94f2 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HBASE/configuration/hbase-env.xml d02ec1b 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HDFS/configuration/hadoop-env.xml ba704ba 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HDFS/package/scripts/params.py 58f01f8 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HIVE/configuration/hive-env.xml 252aa81 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HIVE/configuration/hive-site.xml fabcdf3 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/MAPREDUCE/configuration/mapred-env.xml 2553025 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/NAGIOS/configuration/nagios-env.xml 54c742c 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/OOZIE/configuration/oozie-env.xml 4585f29 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/OOZIE/configuration/oozie-site.xml ae22d5d 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/SQOOP/configuration/sqoop-env.xml f1753c6 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/ZOOKEEPER/configuration/zookeeper-env.xml 33e9cef 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/configuration/cluster-env.xml a2df35a 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-INSTALL/scripts/params.py f76cc33 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-INSTALL/scripts/shared_initialization.py 0542a3e 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/FLUME/configuration/flume-env.xml 43cea5f 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/GANGLIA/configuration/ganglia-env.xml d340a94 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HBASE/configuration/hbase-env.xml 6d27511 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HDFS/configuration/hadoop-env.xml 573a67c 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HDFS/package/scripts/params.py 5c2f792 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HIVE/configuration/hive-env.xml 8c17086 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HIVE/configuration/hive-site.xml 35758fe 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/NAGIOS/configuration/nagios-env.xml 54c742c 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/OOZIE/configuration/oozie-env.xml b1f9090 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/OOZIE/configuration/oozie-site.xml 632d606 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/SQOOP/configuration/sqoop-env.xml d1d56ff 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/YARN/configuration-mapred/mapred-env.xml f0b0ea6 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/YARN/configuration/yarn-env.xml 184aff1 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/ZOOKEEPER/configuration/zookeeper-env.xml 33e9cef 
  ambari-server/src/main/resources/stacks/HDP/2.1.GlusterFS/services/HIVE/configuration/hive-site.xml 36c6985 
  ambari-server/src/main/resources/stacks/HDP/2.1/services/FALCON/configuration/falcon-env.xml 8a4eabf 
  ambari-server/src/main/resources/stacks/HDP/2.1/services/HIVE/configuration/hive-site.xml ae874e2 
  ambari-server/src/main/resources/stacks/HDP/2.1/services/STORM/configuration/storm-env.xml 2e118d1 
  ambari-server/src/main/resources/stacks/HDP/2.1/services/TEZ/configuration/tez-env.xml 992a4df 
  ambari-server/src/main/resources/stacks/HDP/2.1/services/YARN/configuration/yarn-env.xml 4f9b39a 
  ambari-server/src/test/java/org/apache/ambari/server/api/services/AmbariMetaInfoTest.java c08534c 
  ambari-server/src/test/java/org/apache/ambari/server/api/util/StackExtensionHelperTest.java c7af864 
  ambari-server/src/test/java/org/apache/ambari/server/controller/internal/ClusterResourceProviderTest.java 06d8bb7 
  ambari-server/src/test/java/org/apache/ambari/server/orm/entities/BlueprintEntityTest.java de104e7 
  ambari-server/src/test/resources/stacks/HDP/2.0.1/services/HIVE/configuration/hive-site.xml 470f341 

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


Testing
-------

mvn clean test


Thanks,

Andrew Onischuk


Re: Review Request 25256: Stack Service Pluggability: Adding service to a stack requires editing hooks folder

Posted by Dmitro Lisnichenko <dl...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25256/#review52051
-----------------------------------------------------------

Ship it!


Ship It!

- Dmitro Lisnichenko


On Sept. 2, 2014, 5:02 p.m., Andrew Onischuk wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25256/
> -----------------------------------------------------------
> 
> (Updated Sept. 2, 2014, 5:02 p.m.)
> 
> 
> Review request for Ambari, Dmitro Lisnichenko and Vitalyi Brodetskyi.
> 
> 
> Bugs: AMBARI-7115
>     https://issues.apache.org/jira/browse/AMBARI-7115
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Following files has service specific references.
> 
>   1. hooks->BEFORE-INSTALL->shared_initialization.py
>   2. hooks->BEFORE-START->shared_initialization.py
> 
> hooks->BEFORE-INSTALL->shared_initialization.py file is required to change
> while adding a new service package because user creation logic for a service
> resides in the hooks folder.
> 
> Any tasks that needs to be done before installing or starting a service should
> be implemented in a service package.
> 
> **On accomplishing this ticket, It will be possible to plug a new service definition into a deployed cluster without changing any existing python code.**
> 
> 
> Diffs
> -----
> 
>   ambari-server/src/main/java/org/apache/ambari/server/agent/ExecutionCommand.java 33c5af4 
>   ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementControllerImpl.java efce9bd 
>   ambari-server/src/main/java/org/apache/ambari/server/controller/StackConfigurationResponse.java 96ca232 
>   ambari-server/src/main/java/org/apache/ambari/server/controller/StackLevelConfigurationResponse.java 7229197 
>   ambari-server/src/main/java/org/apache/ambari/server/controller/internal/BlueprintResourceProvider.java 2cf01ff 
>   ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ClientConfigResourceProvider.java e5b6016 
>   ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ClusterResourceProvider.java eeee694 
>   ambari-server/src/main/java/org/apache/ambari/server/controller/internal/StackConfigurationResourceProvider.java 2e2bfe1 
>   ambari-server/src/main/java/org/apache/ambari/server/controller/internal/StackLevelConfigurationResourceProvider.java 116f589 
>   ambari-server/src/main/java/org/apache/ambari/server/orm/entities/BlueprintEntity.java 7926fd4 
>   ambari-server/src/main/java/org/apache/ambari/server/state/ConfigHelper.java 43838d7 
>   ambari-server/src/main/java/org/apache/ambari/server/state/PropertyInfo.java 6a17e9d 
>   ambari-server/src/main/resources/properties.json 741ffe6 
>   ambari-server/src/main/resources/stacks/HDP/1.3.2/configuration/cluster-env.xml a2df35a 
>   ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/before-INSTALL/scripts/params.py 2bcbd50 
>   ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/before-INSTALL/scripts/shared_initialization.py f556335 
>   ambari-server/src/main/resources/stacks/HDP/1.3.2/services/GANGLIA/configuration/ganglia-env.xml 68d94f2 
>   ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HBASE/configuration/hbase-env.xml d02ec1b 
>   ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HDFS/configuration/hadoop-env.xml ba704ba 
>   ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HDFS/package/scripts/params.py 58f01f8 
>   ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HIVE/configuration/hive-env.xml 252aa81 
>   ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HIVE/configuration/hive-site.xml fabcdf3 
>   ambari-server/src/main/resources/stacks/HDP/1.3.2/services/MAPREDUCE/configuration/mapred-env.xml 2553025 
>   ambari-server/src/main/resources/stacks/HDP/1.3.2/services/NAGIOS/configuration/nagios-env.xml 54c742c 
>   ambari-server/src/main/resources/stacks/HDP/1.3.2/services/OOZIE/configuration/oozie-env.xml 4585f29 
>   ambari-server/src/main/resources/stacks/HDP/1.3.2/services/OOZIE/configuration/oozie-site.xml ae22d5d 
>   ambari-server/src/main/resources/stacks/HDP/1.3.2/services/SQOOP/configuration/sqoop-env.xml f1753c6 
>   ambari-server/src/main/resources/stacks/HDP/1.3.2/services/ZOOKEEPER/configuration/zookeeper-env.xml 33e9cef 
>   ambari-server/src/main/resources/stacks/HDP/2.0.6/configuration/cluster-env.xml a2df35a 
>   ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-INSTALL/scripts/params.py f76cc33 
>   ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-INSTALL/scripts/shared_initialization.py 0542a3e 
>   ambari-server/src/main/resources/stacks/HDP/2.0.6/services/FLUME/configuration/flume-env.xml 43cea5f 
>   ambari-server/src/main/resources/stacks/HDP/2.0.6/services/GANGLIA/configuration/ganglia-env.xml d340a94 
>   ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HBASE/configuration/hbase-env.xml 6d27511 
>   ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HDFS/configuration/hadoop-env.xml 573a67c 
>   ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HDFS/package/scripts/params.py 5c2f792 
>   ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HIVE/configuration/hive-env.xml 8c17086 
>   ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HIVE/configuration/hive-site.xml 35758fe 
>   ambari-server/src/main/resources/stacks/HDP/2.0.6/services/NAGIOS/configuration/nagios-env.xml 54c742c 
>   ambari-server/src/main/resources/stacks/HDP/2.0.6/services/OOZIE/configuration/oozie-env.xml b1f9090 
>   ambari-server/src/main/resources/stacks/HDP/2.0.6/services/OOZIE/configuration/oozie-site.xml 632d606 
>   ambari-server/src/main/resources/stacks/HDP/2.0.6/services/SQOOP/configuration/sqoop-env.xml d1d56ff 
>   ambari-server/src/main/resources/stacks/HDP/2.0.6/services/YARN/configuration-mapred/mapred-env.xml f0b0ea6 
>   ambari-server/src/main/resources/stacks/HDP/2.0.6/services/YARN/configuration/yarn-env.xml 184aff1 
>   ambari-server/src/main/resources/stacks/HDP/2.0.6/services/ZOOKEEPER/configuration/zookeeper-env.xml 33e9cef 
>   ambari-server/src/main/resources/stacks/HDP/2.1.GlusterFS/services/HIVE/configuration/hive-site.xml 36c6985 
>   ambari-server/src/main/resources/stacks/HDP/2.1/services/FALCON/configuration/falcon-env.xml 8a4eabf 
>   ambari-server/src/main/resources/stacks/HDP/2.1/services/HIVE/configuration/hive-site.xml ae874e2 
>   ambari-server/src/main/resources/stacks/HDP/2.1/services/STORM/configuration/storm-env.xml 2e118d1 
>   ambari-server/src/main/resources/stacks/HDP/2.1/services/TEZ/configuration/tez-env.xml 992a4df 
>   ambari-server/src/main/resources/stacks/HDP/2.1/services/YARN/configuration/yarn-env.xml 4f9b39a 
>   ambari-server/src/test/java/org/apache/ambari/server/api/services/AmbariMetaInfoTest.java c08534c 
>   ambari-server/src/test/java/org/apache/ambari/server/api/util/StackExtensionHelperTest.java c7af864 
>   ambari-server/src/test/java/org/apache/ambari/server/controller/internal/ClusterResourceProviderTest.java 06d8bb7 
>   ambari-server/src/test/java/org/apache/ambari/server/orm/entities/BlueprintEntityTest.java de104e7 
>   ambari-server/src/test/resources/stacks/HDP/2.0.1/services/HIVE/configuration/hive-site.xml 470f341 
> 
> Diff: https://reviews.apache.org/r/25256/diff/
> 
> 
> Testing
> -------
> 
> mvn clean test
> 
> 
> Thanks,
> 
> Andrew Onischuk
> 
>


Re: Review Request 25256: Stack Service Pluggability: Adding service to a stack requires editing hooks folder

Posted by Andrew Onischuk <ao...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25256/
-----------------------------------------------------------

(Updated Sept. 2, 2014, 5:02 p.m.)


Review request for Ambari, Dmitro Lisnichenko and Vitalyi Brodetskyi.


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


Repository: ambari


Description
-------

Following files has service specific references.

  1. hooks->BEFORE-INSTALL->shared_initialization.py
  2. hooks->BEFORE-START->shared_initialization.py

hooks->BEFORE-INSTALL->shared_initialization.py file is required to change
while adding a new service package because user creation logic for a service
resides in the hooks folder.

Any tasks that needs to be done before installing or starting a service should
be implemented in a service package.

**On accomplishing this ticket, It will be possible to plug a new service definition into a deployed cluster without changing any existing python code.**


Diffs
-----

  ambari-server/src/main/java/org/apache/ambari/server/agent/ExecutionCommand.java 33c5af4 
  ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementControllerImpl.java efce9bd 
  ambari-server/src/main/java/org/apache/ambari/server/controller/StackConfigurationResponse.java 96ca232 
  ambari-server/src/main/java/org/apache/ambari/server/controller/StackLevelConfigurationResponse.java 7229197 
  ambari-server/src/main/java/org/apache/ambari/server/controller/internal/BlueprintResourceProvider.java 2cf01ff 
  ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ClientConfigResourceProvider.java e5b6016 
  ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ClusterResourceProvider.java eeee694 
  ambari-server/src/main/java/org/apache/ambari/server/controller/internal/StackConfigurationResourceProvider.java 2e2bfe1 
  ambari-server/src/main/java/org/apache/ambari/server/controller/internal/StackLevelConfigurationResourceProvider.java 116f589 
  ambari-server/src/main/java/org/apache/ambari/server/orm/entities/BlueprintEntity.java 7926fd4 
  ambari-server/src/main/java/org/apache/ambari/server/state/ConfigHelper.java 43838d7 
  ambari-server/src/main/java/org/apache/ambari/server/state/PropertyInfo.java 6a17e9d 
  ambari-server/src/main/resources/properties.json 741ffe6 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/configuration/cluster-env.xml a2df35a 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/before-INSTALL/scripts/params.py 2bcbd50 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/before-INSTALL/scripts/shared_initialization.py f556335 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/GANGLIA/configuration/ganglia-env.xml 68d94f2 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HBASE/configuration/hbase-env.xml d02ec1b 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HDFS/configuration/hadoop-env.xml ba704ba 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HDFS/package/scripts/params.py 58f01f8 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HIVE/configuration/hive-env.xml 252aa81 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HIVE/configuration/hive-site.xml fabcdf3 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/MAPREDUCE/configuration/mapred-env.xml 2553025 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/NAGIOS/configuration/nagios-env.xml 54c742c 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/OOZIE/configuration/oozie-env.xml 4585f29 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/OOZIE/configuration/oozie-site.xml ae22d5d 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/SQOOP/configuration/sqoop-env.xml f1753c6 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/ZOOKEEPER/configuration/zookeeper-env.xml 33e9cef 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/configuration/cluster-env.xml a2df35a 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-INSTALL/scripts/params.py f76cc33 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-INSTALL/scripts/shared_initialization.py 0542a3e 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/FLUME/configuration/flume-env.xml 43cea5f 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/GANGLIA/configuration/ganglia-env.xml d340a94 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HBASE/configuration/hbase-env.xml 6d27511 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HDFS/configuration/hadoop-env.xml 573a67c 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HDFS/package/scripts/params.py 5c2f792 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HIVE/configuration/hive-env.xml 8c17086 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HIVE/configuration/hive-site.xml 35758fe 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/NAGIOS/configuration/nagios-env.xml 54c742c 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/OOZIE/configuration/oozie-env.xml b1f9090 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/OOZIE/configuration/oozie-site.xml 632d606 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/SQOOP/configuration/sqoop-env.xml d1d56ff 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/YARN/configuration-mapred/mapred-env.xml f0b0ea6 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/YARN/configuration/yarn-env.xml 184aff1 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/ZOOKEEPER/configuration/zookeeper-env.xml 33e9cef 
  ambari-server/src/main/resources/stacks/HDP/2.1.GlusterFS/services/HIVE/configuration/hive-site.xml 36c6985 
  ambari-server/src/main/resources/stacks/HDP/2.1/services/FALCON/configuration/falcon-env.xml 8a4eabf 
  ambari-server/src/main/resources/stacks/HDP/2.1/services/HIVE/configuration/hive-site.xml ae874e2 
  ambari-server/src/main/resources/stacks/HDP/2.1/services/STORM/configuration/storm-env.xml 2e118d1 
  ambari-server/src/main/resources/stacks/HDP/2.1/services/TEZ/configuration/tez-env.xml 992a4df 
  ambari-server/src/main/resources/stacks/HDP/2.1/services/YARN/configuration/yarn-env.xml 4f9b39a 
  ambari-server/src/test/java/org/apache/ambari/server/api/services/AmbariMetaInfoTest.java c08534c 
  ambari-server/src/test/java/org/apache/ambari/server/api/util/StackExtensionHelperTest.java c7af864 
  ambari-server/src/test/java/org/apache/ambari/server/controller/internal/ClusterResourceProviderTest.java 06d8bb7 
  ambari-server/src/test/java/org/apache/ambari/server/orm/entities/BlueprintEntityTest.java de104e7 
  ambari-server/src/test/resources/stacks/HDP/2.0.1/services/HIVE/configuration/hive-site.xml 470f341 

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


Testing
-------

mvn clean test


Thanks,

Andrew Onischuk