----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59948/#review177500 -----------------------------------------------------------
ambari-server/src/main/java/org/apache/ambari/server/api/services/stackadvisor/StackAdvisorHelper.java Line 78 (original), 87 (patched) <https://reviews.apache.org/r/59948/#comment251096> Ok to do this for now since StackAdvisor runs on all services today anyways and all services are still defaulting to Python. I've created a separate Jira for the UI to pass in the service name. ambari-server/src/main/java/org/apache/ambari/server/stack/ServiceModule.java Lines 210 (patched) <https://reviews.apache.org/r/59948/#comment251097> Default to Python pom.xml Lines 462 (patched) <https://reviews.apache.org/r/59948/#comment251095> New module to handle class loading and also compile independently. - Alejandro Fernandez On June 9, 2017, 5:57 p.m., Alejandro Fernandez wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/59948/ > ----------------------------------------------------------- > > (Updated June 9, 2017, 5:57 p.m.) > > > Review request for Ambari, Jonathan Hurley, Nate Cole, Sumit Mohanty, Swapan > Shridhar, and Sid Wagle. > > > Bugs: AMBARI-20853 > https://issues.apache.org/jira/browse/AMBARI-20853 > > > Repository: ambari > > > Description > ------- > > See Epic AMBARI-20852 > > First step is to allow a Service's metainfo.xml file to define what type of > Service Advisor it uses, Python or Java. > For now, continue to call Python for all services since all services are not > specifying which type to use (so defaults to Python) > > If set to Java, will eventually call a class to invoke the existing Service > Advisor in Python as a way to ensure compatibility. > If set to Python, continue to call the current script. > > In both cases, we will continue to generate the services.json file. > > Today, we call Stack Advisor on all services even if only one service is > modified, so it's ok to hardcode a service name until the UI is able to > provide that granularity. > > > Diffs > ----- > > ambari-server/checkstyle.xml 6b4824a > ambari-server/pom.xml c0aed7b > > ambari-server/src/main/java/org/apache/ambari/server/api/services/stackadvisor/StackAdvisorHelper.java > 3083d9f > > ambari-server/src/main/java/org/apache/ambari/server/api/services/stackadvisor/StackAdvisorRunner.java > a4b7915 > > ambari-server/src/main/java/org/apache/ambari/server/api/services/stackadvisor/commands/ComponentLayoutRecommendationCommand.java > ca1d95f2 > > ambari-server/src/main/java/org/apache/ambari/server/api/services/stackadvisor/commands/ComponentLayoutValidationCommand.java > 91d8f9c > > ambari-server/src/main/java/org/apache/ambari/server/api/services/stackadvisor/commands/ConfigurationDependenciesRecommendationCommand.java > fae0b16 > > ambari-server/src/main/java/org/apache/ambari/server/api/services/stackadvisor/commands/ConfigurationRecommendationCommand.java > e25e39b > > ambari-server/src/main/java/org/apache/ambari/server/api/services/stackadvisor/commands/ConfigurationValidationCommand.java > 1892a82 > > ambari-server/src/main/java/org/apache/ambari/server/api/services/stackadvisor/commands/StackAdvisorCommand.java > ed27697 > > ambari-server/src/main/java/org/apache/ambari/server/stack/ServiceModule.java > 7dbc639 > ambari-server/src/main/java/org/apache/ambari/server/state/ServiceInfo.java > 97490e0 > > ambari-server/src/test/java/org/apache/ambari/server/api/services/stackadvisor/StackAdvisorHelperTest.java > e331ddd > > ambari-server/src/test/java/org/apache/ambari/server/api/services/stackadvisor/StackAdvisorRunnerTest.java > 8d104ea > > ambari-server/src/test/java/org/apache/ambari/server/api/services/stackadvisor/commands/ConfigurationRecommendationCommandTest.java > 17293eb > > ambari-server/src/test/java/org/apache/ambari/server/api/services/stackadvisor/commands/StackAdvisorCommandTest.java > 46552a3 > > ambari-server/src/test/java/org/apache/ambari/server/stack/ServiceModuleTest.java > 20d2168 > pom.xml 8faec29 > serviceadvisor/pom.xml PRE-CREATION > > serviceadvisor/src/main/java/org/apache/ambari/serviceadvisor/ServiceAdvisor.java > PRE-CREATION > > serviceadvisor/src/main/java/org/apache/ambari/serviceadvisor/ServiceAdvisorCommandType.java > PRE-CREATION > > > Diff: https://reviews.apache.org/r/59948/diff/1/ > > > Testing > ------- > > Waiting for unit test results. > Verified on a live cluster. > > > Thanks, > > Alejandro Fernandez > >
