----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55817/#review166416 -----------------------------------------------------------
ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/metainfo.xml (line 109) <https://reviews.apache.org/r/55817/#comment238346> Hello Yao, As you said in comment to Matt's question on cross-service co-hosting, your change is currently just to move the logic from hardcode logic on UI to the backend. I wonder if this can be improved a little bit to support multiple components. All components should be from the same service for the current design. (I think stack_advisor.py handles as cross-service co host by part of the laytout calculation.) the coHost section in metainfo.xml files can be <coHost> <component>A1</component> <component>A2</component> </coHost> UI code needs updates accordingly. - Di Li On Feb. 22, 2017, 1:52 a.m., yao lei wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/55817/ > ----------------------------------------------------------- > > (Updated Feb. 22, 2017, 1:52 a.m.) > > > Review request for Ambari, Alejandro Fernandez, Di Li, Jaimin Jetly, Jayush > Luniya, Zhe (Joe) Wang, and Matt. > > > Bugs: AMBARI-19618 > https://issues.apache.org/jira/browse/AMBARI-19618 > > > Repository: ambari > > > Description > ------- > > Cohosted components information now is hardcoded in > stack_service_component.js as following: > > App.StackServiceComponent.coHost = > { 'WEBHCAT_SERVER': 'HIVE_SERVER' }; > > It's better to move them from javascript to metainfom.xml by adding tag > <coHost></coHost>in stack like following: > > //HIVE > <component> > <name>WEBHCAT_SERVER</name> > <displayName>WebHCat Server</displayName> > <category>MASTER</category> > <cardinality>1</cardinality> > <versionAdvertised>true</versionAdvertised> > <reassignAllowed>true</reassignAllowed> > <coHost>HIVE_SERVER</coHost> > </component> > > //RANGER https://issues.apache.org/jira/browse/AMBARI-19557 > <component> > <name>RANGER_USERSYNC</name> > <displayName>Ranger Usersync</displayName> > <category>MASTER</category> > <cardinality>1</cardinality> > <versionAdvertised>true</versionAdvertised> > <coHost>RANGER_ADMIN</coHost> > </component> > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/controller/StackServiceComponentResponse.java > 2f42313 > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/StackServiceComponentResourceProvider.java > c731641 > > ambari-server/src/main/java/org/apache/ambari/server/stack/ComponentModule.java > 51d9847 > > ambari-server/src/main/java/org/apache/ambari/server/state/ComponentInfo.java > 409bcae > > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/metainfo.xml > 4ba3cf1 > ambari-server/src/main/resources/common-services/RANGER/0.4.0/metainfo.xml > a9db470 > ambari-server/src/main/resources/properties.json 72104a9 > > ambari-server/src/test/java/org/apache/ambari/server/controller/AmbariManagementControllerTest.java > 89f9d94 > > ambari-server/src/test/java/org/apache/ambari/server/stack/ComponentModuleTest.java > 6e37ded > > ambari-server/src/test/java/org/apache/ambari/server/stack/StackManagerTest.java > 87a1fc7 > > ambari-server/src/test/resources/stacks/HDP/2.0.5/services/HIVE/metainfo.xml > 3224eac > > ambari-server/src/test/resources/stacks/HDP/2.0.5/services/RANGER/metainfo.xml > PRE-CREATION > > ambari-server/src/test/resources/stacks/HDP/2.0.6/services/HIVE/metainfo.xml > edc5dfb > > ambari-server/src/test/resources/stacks/HDP/2.0.6/services/RANGER/metainfo.xml > PRE-CREATION > ambari-web/app/mappers/stack_service_mapper.js 4bda89d > ambari-web/app/models/stack_service_component.js eb6f2db > ambari-web/test/mappers/stack_service_mapper_test.js 9da8b24 > ambari-web/test/service_components.js bcc4a29 > > Diff: https://reviews.apache.org/r/55817/diff/ > > > Testing > ------- > > 1.Ambari Web Unit Tests: > 20307 passing (35s) > 153 pending > > 2.Ambari Server Unit Tests: > Failed tests: > ViewRegistryTest.testReadViewArchives:239->testReadViewArchives:466 > expected:<DEPLOYED> but was:<ERROR> > > ViewRegistryTest.testReadViewArchives_removeUndeployed:244->testReadViewArchives:466 > expected:<DEPLOYED> but was:<ERROR> > > ViewRegistryTest.testReadViewArchives_viewAutoInstanceCreation:254->testReadViewArchives:466 > expected:<DEPLOYED> but was:<ERROR> > Tests run: 4417, Failures: 3, Errors: 0, Skipped: 30 > > 3.Installed a cluster using RPM based on latest trunk codes and screenshots > attached in https://issues.apache.org/jira/browse/AMBARI-19618 show the > expected result. > > > Thanks, > > yao lei > >