-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20501/
-----------------------------------------------------------
(Updated April 23, 2014, 6:33 p.m.)
Review request for Ambari, Jaimin Jetly, John Speidel, Sumit Mohanty, and
Srimanth Gunturi.
Changes
-------
- New patch without any unit tests, need some feedback on the approach.
- The stack definition will mark properties as being required, if blueprint
definition does not have these properties, we error out on Blueprint creation.
- Passwords are special properties since we need to provide a default is none
provided, the type property on a configuration qualifies a password.
- Default password is service in all lowercase.
Bugs: AMBARI-5515
https://issues.apache.org/jira/browse/AMBARI-5515
Repository: ambari
Description
-------
Hive deployment via Blueprints works when the following configurations were
added to the blueprint.
Several of the properties do exist in the stack definition, but most don't.
Diffs (updated)
-----
ambari-server/src/main/java/org/apache/ambari/server/api/services/AmbariMetaInfo.java
659499e
ambari-server/src/main/java/org/apache/ambari/server/api/util/StackExtensionHelper.java
08a545f
ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariServer.java
034e877
ambari-server/src/main/java/org/apache/ambari/server/controller/StackConfigurationResponse.java
e5cbdf7
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/BlueprintResourceProvider.java
ea2fdec
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ClusterResourceProvider.java
18031d0
ambari-server/src/main/java/org/apache/ambari/server/state/PropertyInfo.java
acc5f4a
ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HIVE/configuration/global.xml
f3c274a
ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HIVE/configuration/hive-site.xml
3a6ed76
ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HIVE/configuration/global.xml
e164c64
ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HIVE/configuration/hive-site.xml
6336a70
ambari-server/src/main/resources/stacks/HDP/2.0.6/services/NAGIOS/configuration/global.xml
61a2b90
ambari-server/src/main/resources/stacks/HDP/2.1/services/HIVE/configuration/hive-site.xml
781fdcb
ambari-server/src/test/java/org/apache/ambari/server/api/services/AmbariMetaInfoTest.java
3a63b78
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/BlueprintResourceProviderTest.java
9e12c63
ambari-server/src/test/resources/stacks/HDP/2.0.1/services/HIVE/configuration/hive-site.xml
7d35558
Diff: https://reviews.apache.org/r/20501/diff/
Testing
-------
mvn clean test -Djava.awt.headless=true -DfailIfNoTests=false
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO]
[INFO] Ambari Main ....................................... SUCCESS [1.966s]
[INFO] Apache Ambari Project POM ......................... SUCCESS [0.303s]
[INFO] Ambari Web ........................................ SUCCESS [9.690s]
[INFO] Ambari Views ...................................... SUCCESS [1.835s]
[INFO] Ambari Server ..................................... SUCCESS [14:38.002s]
[INFO] Ambari Agent ...................................... SUCCESS [15.533s]
[INFO] Ambari Client ..................................... SUCCESS [0.522s]
[INFO] ------------------------------------------------------------------------
Thanks,
Sid Wagle