-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/58845/
-----------------------------------------------------------
(Updated May 5, 2017, 6:26 p.m.)
Review request for Ambari, Alejandro Fernandez, Di Li, Jonathan Hurley, Jayush
Luniya, and Nate Cole.
Changes
-------
Fixed issue reported by Jonathan Hurley
Bugs: AMBARI-20891
https://issues.apache.org/jira/browse/AMBARI-20891
Repository: ambari
Description
-------
It would possible to link extensions to supported stack versions while parsing
the stacks, extensions and common-services directories.
This would allow extensions to avoid making rest API calls to set up the link.
Diffs (updated)
-----
ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementControllerImpl.java
751ce08
ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementHelper.java
PRE-CREATION
ambari-server/src/main/java/org/apache/ambari/server/orm/dao/ExtensionLinkDAO.java
3df1cd5
ambari-server/src/main/java/org/apache/ambari/server/orm/entities/ExtensionLinkEntity.java
12b3ce0
ambari-server/src/main/java/org/apache/ambari/server/stack/ExtensionModule.java
ce92ecf
ambari-server/src/main/java/org/apache/ambari/server/stack/StackManager.java
20e28fc
ambari-server/src/main/java/org/apache/ambari/server/stack/StackModule.java
d3ad351
ambari-server/src/main/java/org/apache/ambari/server/state/ExtensionInfo.java
656a8ce
ambari-server/src/main/java/org/apache/ambari/server/state/StackInfo.java
353dd86
ambari-server/src/main/java/org/apache/ambari/server/state/stack/ExtensionMetainfoXml.java
77a7250
ambari-server/src/test/java/org/apache/ambari/server/stack/StackManagerCommonServicesTest.java
4e20c2b
ambari-server/src/test/java/org/apache/ambari/server/stack/StackManagerExtensionTest.java
9900eb5
ambari-server/src/test/java/org/apache/ambari/server/stack/StackManagerMiscTest.java
ca24cd9
ambari-server/src/test/java/org/apache/ambari/server/stack/StackManagerMock.java
0fe1573
ambari-server/src/test/java/org/apache/ambari/server/stack/StackManagerTest.java
d8c02dc
ambari-server/src/test/resources/extensions/EXT/0.1/metainfo.xml 77a832c
ambari-server/src/test/resources/extensions/EXT/0.2/metainfo.xml 04f733c
ambari-server/src/test/resources/extensions/EXT/0.3/metainfo.xml PRE-CREATION
ambari-server/src/test/resources/extensions/EXT/0.3/services/OOZIE2/metainfo.xml
PRE-CREATION
ambari-server/src/test/resources/extensions/EXT/0.3/services/OOZIE2/themes/broken_theme.json
PRE-CREATION
ambari-server/src/test/resources/stacks_with_extensions/HDP/0.3/metainfo.xml
PRE-CREATION
ambari-server/src/test/resources/stacks_with_extensions/HDP/0.3/repos/repoinfo.xml
PRE-CREATION
ambari-server/src/test/resources/stacks_with_extensions/HDP/0.3/services/HBASE/metainfo.xml
PRE-CREATION
ambari-server/src/test/resources/stacks_with_extensions/HDP/0.3/services/HDFS/configuration/global.xml
PRE-CREATION
ambari-server/src/test/resources/stacks_with_extensions/HDP/0.3/services/HDFS/configuration/hadoop-env.xml
PRE-CREATION
ambari-server/src/test/resources/stacks_with_extensions/HDP/0.3/services/HDFS/configuration/hbase-site.xml
PRE-CREATION
ambari-server/src/test/resources/stacks_with_extensions/HDP/0.3/services/HDFS/configuration/hdfs-log4j.xml
PRE-CREATION
ambari-server/src/test/resources/stacks_with_extensions/HDP/0.3/services/HDFS/configuration/hdfs-site.xml
PRE-CREATION
ambari-server/src/test/resources/stacks_with_extensions/HDP/0.3/services/HDFS/metainfo.xml
PRE-CREATION
ambari-server/src/test/resources/stacks_with_extensions/HDP/0.3/services/HDFS/package/dummy-script.py
PRE-CREATION
ambari-server/src/test/resources/stacks_with_extensions/HDP/0.3/services/HIVE/metainfo.xml
PRE-CREATION
ambari-server/src/test/resources/stacks_with_extensions/HDP/0.3/services/MAPREDUCE/metainfo.xml
PRE-CREATION
ambari-server/src/test/resources/stacks_with_extensions/HDP/0.3/services/ZOOKEEPER/metainfo.xml
PRE-CREATION
Diff: https://reviews.apache.org/r/58845/diff/2/
Changes: https://reviews.apache.org/r/58845/diff/1-2/
Testing
-------
mvn clean test
mvn clean test -Dtest=StackManagerExtensionTest -DskipPythonTests
Thanks,
Tim Thorpe
