> On Nov. 29, 2016, 6:53 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/STORM/0.10.0/configuration/storm-env.xml, > > line 32 > > <https://reviews.apache.org/r/54169/diff/1/?file=1572153#file1572153line32> > > > > This property will be added automatically after an Ambari Upgrade with > > the value "false". Should this be smarter to detect if Atlas and Storm are > > in the cluster and if so then set the value to true? > > Mugdha Varadkar wrote: > Hi Alejandro, > Yes there is a way this can be handled in UpgradeCatalog250.java. This > fix will is getting handled in > https://issues.apache.org/jira/browse/AMBARI-19024 > > Alejandro Fernandez wrote: > Mugdha, a lot of the items I brought up are being deferred to another > Jira. In my opinion, they should be done as part of this commit. Typically, > it's ok to create another Jira for additional functionality, but not very > common to fix more issues with the current patch. Thanks.
Updated in latest patch - Mugdha ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54169/#review157296 ----------------------------------------------------------- On Dec. 1, 2016, 6:59 a.m., Mugdha Varadkar wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/54169/ > ----------------------------------------------------------- > > (Updated Dec. 1, 2016, 6:59 a.m.) > > > Review request for Ambari, Alejandro Fernandez, Gautam Borad, Madhan > Neethiraj, and Sumit Mohanty. > > > Bugs: AMBARI-19007 > https://issues.apache.org/jira/browse/AMBARI-19007 > > > Repository: ambari > > > Description > ------- > > 1) Introduced new flag for all the hook suported > components(HIVE,STORM,SQOOP,FALCON)under there respective env config-type. > 2) If ATLAS service is present/selected for install, stack-advisor will set > the hook flag. This flag value is used to recommend the expected > configuration required for hook to work. > 3) For Blue-print based installation user need to set hook flag and add > common-atlas application properties under each hook config-type eg: > hive-atlas-application.properties config-type which are require to > communicate with external ATLAS > > > Diffs > ----- > > > ambari-common/src/main/python/resource_management/libraries/functions/setup_atlas_hook.py > a1d2f95 > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/BlueprintConfigurationProcessor.java > ec846f8 > > ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog250.java > 52de784 > > ambari-server/src/main/resources/common-services/FALCON/0.5.0.2.1/configuration/falcon-env.xml > 09cced6 > > ambari-server/src/main/resources/common-services/FALCON/0.5.0.2.1/package/scripts/falcon.py > 4429253 > > ambari-server/src/main/resources/common-services/FALCON/0.5.0.2.1/package/scripts/params_linux.py > 0fb21d0 > > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/configuration/hive-env.xml > 150f629 > > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/hcat.py > 5e2c709 > > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/hive.py > bcc598a > > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/params_linux.py > 01e5f00 > > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/webhcat.py > fe3f34a > > ambari-server/src/main/resources/common-services/SQOOP/1.4.4.2.0/configuration/sqoop-env.xml > f682e97 > > ambari-server/src/main/resources/common-services/SQOOP/1.4.4.2.0/configuration/sqoop-site.xml > f7823d2 > > ambari-server/src/main/resources/common-services/SQOOP/1.4.4.2.0/package/scripts/params_linux.py > 283f54d > > ambari-server/src/main/resources/common-services/SQOOP/1.4.4.2.0/package/scripts/sqoop.py > 68f06db > > ambari-server/src/main/resources/common-services/STORM/0.10.0/configuration/storm-env.xml > 9547335 > > ambari-server/src/main/resources/common-services/STORM/0.9.1/package/scripts/params_linux.py > e753e98 > > ambari-server/src/main/resources/common-services/STORM/0.9.1/package/scripts/storm.py > bda4fe2 > > ambari-server/src/main/resources/stacks/HDP/2.2/services/HIVE/configuration/hive-site.xml > ab350dc > > ambari-server/src/main/resources/stacks/HDP/2.3/services/STORM/configuration/storm-site.xml > b71f4a9 > ambari-server/src/main/resources/stacks/HDP/2.3/services/stack_advisor.py > 775dbab > > ambari-server/src/test/java/org/apache/ambari/server/controller/internal/BlueprintConfigurationProcessorTest.java > f2dd099 > > ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog250Test.java > ce0b387 > ambari-server/src/test/python/stacks/2.3/common/test_stack_advisor.py > 39cbbce > > Diff: https://reviews.apache.org/r/54169/diff/ > > > Testing > ------- > > Tested Atlas installation via UI and blue-print. With blueprint used > config_recommendation_strategy as NEVER_APPLY and > ALWAYS_APPLY_DONT_OVERRIDE_CUSTOM_VALUES > > > mvn -DskipPythonTests -Dtest=BlueprintConfigurationProcessorTest test > > ------------------------------------------------------- > T E S T S > ------------------------------------------------------- > Picked up _JAVA_OPTIONS: -Xmx2048m -XX:MaxPermSize=512m > -Djava.awt.headless=true > Running > org.apache.ambari.server.controller.internal.BlueprintConfigurationProcessorTest > Tests run: 167, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 1.101 sec - > in > org.apache.ambari.server.controller.internal.BlueprintConfigurationProcessorTest > > Results : > > Tests run: 167, Failures: 0, Errors: 0, Skipped: 0 > > > [INFO] > ------------------------------------------------------------------------ > [INFO] BUILD SUCCESS > [INFO] > ------------------------------------------------------------------------ > [INFO] Total time: 24.484s > [INFO] Finished at: Tue Nov 29 11:40:30 UTC 2016 > [INFO] Final Memory: 62M/797M > [INFO] > ------------------------------------------------------------------------ > > > Thanks, > > Mugdha Varadkar > >