> On April 15, 2014, 6:30 p.m., John Speidel wrote: > > ambari-server/src/main/java/org/apache/ambari/server/api/util/StackExtensionHelper.java, > > line 597 > > <https://reviews.apache.org/r/20346/diff/1/?file=557721#file557721line597> > > > > Why would the property name be null or empty? Was there an issue which > > prompted adding this check? > > Sid Wagle wrote: > Correct, there was empty property and it resulted in an UI issue.
Wouldn't it be better to remove the offending property from the stack? How is a property with no name useful? > On April 15, 2014, 6:30 p.m., John Speidel wrote: > > ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HBASE/configuration/global.xml, > > lines 25-34 > > <https://reviews.apache.org/r/20346/diff/1/?file=557722#file557722line25> > > > > Why are we deleting properties which previously existed but had no > > values? Are these properties no longer used? This comment applies to all > > properties which were removed across all of the diffs. > > > > There are some deleted properties which seem that they are still used > > and would have valid default values such as hbase.master.info.port ... > > > > > > Sid Wagle wrote: > The deleted properties are of the form "_host(s)". These are used to > convey cluster topology info to the agent. The server sends these without > looking at the stack properties so these were pretty much placeholder with > current code. > > Another point is, if someone wants to add a Service we cannot rely on > topology related properties to be present in the global config of a new > service definition. > > I have opened a new Jira for standardizing on the way topology info is > sent, basically something like, componentName_category_host(s). ok. But why would be delete properties such as hbase-site/hbase.master.info.port? - John ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20346/#review40418 ----------------------------------------------------------- On April 15, 2014, 7:35 p.m., Sid Wagle wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/20346/ > ----------------------------------------------------------- > > (Updated April 15, 2014, 7:35 p.m.) > > > Review request for Ambari, John Speidel, Sumit Mohanty, and Yusaku Sako. > > > Bugs: AMBARI-5387 > https://issues.apache.org/jira/browse/AMBARI-5387 > > > Repository: ambari > > > Description > ------- > > In host overrides we depend on properties defined in {{global.xml}} to > determine which service needs to be restarted when properties are changed. > > The stack definition API we call however does not provide properties with > empty values in {{global.xml}}. This effects UI because we do not show a > service restart required when necessary. > > http://server:8080/api/v1/stacks2/HDP/versions/2.0.6/stackServices?fields=configurations/StackConfigurations/type > > > The specific property was HBase's {{hregion_memstoreflushsize}}. > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/api/services/AmbariMetaInfo.java > 52c7309 > > ambari-server/src/main/java/org/apache/ambari/server/api/util/StackExtensionHelper.java > c31d437 > > ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HBASE/configuration/global.xml > 453184b > > ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HBASE/configuration/hbase-site.xml > bd4d61f > > ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HDFS/configuration/core-site.xml > d2bff0f > > ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HDFS/configuration/global.xml > 04d51db > > ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HDFS/configuration/hadoop-policy.xml > 900da99 > > ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HDFS/configuration/hdfs-site.xml > 9c28bec > > ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HIVE/configuration/global.xml > ae7f586 > > ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HIVE/configuration/hive-site.xml > 34e6231 > > ambari-server/src/main/resources/stacks/HDP/1.3.2/services/MAPREDUCE/configuration/global.xml > 4633855 > > ambari-server/src/main/resources/stacks/HDP/1.3.2/services/MAPREDUCE/configuration/mapred-site.xml > 7255ca3 > > ambari-server/src/main/resources/stacks/HDP/1.3.2/services/OOZIE/configuration/global.xml > ddbf780 > > ambari-server/src/main/resources/stacks/HDP/1.3.2/services/OOZIE/configuration/oozie-site.xml > f4d4b63 > > ambari-server/src/main/resources/stacks/HDP/1.3.2/services/WEBHCAT/configuration/webhcat-site.xml > 16d8691 > > ambari-server/src/main/resources/stacks/HDP/1.3.2/services/ZOOKEEPER/configuration/global.xml > f78df89 > > ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HBASE/configuration/global.xml > b2c57bd > > ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HBASE/configuration/hbase-site.xml > cf9416e > > ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HDFS/configuration/global.xml > ffda6e2 > > ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HDFS/configuration/hdfs-site.xml > d7b37c4 > > ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HIVE/configuration/global.xml > ae7f586 > > ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HIVE/configuration/hive-site.xml > 1e4ba38 > > ambari-server/src/main/resources/stacks/HDP/2.0.6/services/OOZIE/configuration/global.xml > ddbf780 > > ambari-server/src/main/resources/stacks/HDP/2.0.6/services/OOZIE/configuration/oozie-site.xml > f96e562 > > ambari-server/src/main/resources/stacks/HDP/2.0.6/services/WEBHCAT/configuration/webhcat-site.xml > 5b5115a > > ambari-server/src/main/resources/stacks/HDP/2.0.6/services/YARN/configuration-mapred/global.xml > 984cd41 > > ambari-server/src/main/resources/stacks/HDP/2.0.6/services/YARN/configuration-mapred/mapred-site.xml > 310095f > > ambari-server/src/main/resources/stacks/HDP/2.0.6/services/YARN/configuration/global.xml > c3a37ef > > ambari-server/src/main/resources/stacks/HDP/2.0.6/services/ZOOKEEPER/configuration/global.xml > f78df89 > > ambari-server/src/main/resources/stacks/HDP/2.1/services/FALCON/configuration/falcon-startup.properties.xml > 9746e24 > > ambari-server/src/main/resources/stacks/HDP/2.1/services/HIVE/configuration/hive-site.xml > e1e3812 > > ambari-server/src/main/resources/stacks/HDP/2.1/services/OOZIE/configuration/oozie-site.xml > f320b23 > > ambari-server/src/main/resources/stacks/HDP/2.1/services/WEBHCAT/configuration/webhcat-site.xml > 130004a > > ambari-server/src/main/resources/stacks/HDP/2.1/services/YARN/configuration/global.xml > 9c748da > > ambari-server/src/test/java/org/apache/ambari/server/api/services/AmbariMetaInfoTest.java > 8c0bd65 > > ambari-server/src/test/java/org/apache/ambari/server/api/util/StackExtensionHelperTest.java > 2f17c52 > > ambari-server/src/test/java/org/apache/ambari/server/controller/AmbariManagementControllerTest.java > 73530a8 > ambari-web/app/data/HDP2/site_properties.js 3e1c230 > ambari-web/app/utils/helper.js 1335ff2 > ambari-web/app/views/common/configs/services_config.js 8143711 > > Diff: https://reviews.apache.org/r/20346/diff/ > > > Testing > ------- > > mvn clean test -Djava.awt.headless=true -DfailIfNoTests=false > > [INFO] > ------------------------------------------------------------------------ > [INFO] Reactor Summary: > [INFO] > [INFO] Ambari Main ....................................... SUCCESS [1.903s] > [INFO] Apache Ambari Project POM ......................... SUCCESS [0.301s] > [INFO] Ambari Web ........................................ SUCCESS [9.738s] > [INFO] Ambari Views ...................................... SUCCESS [1.824s] > [INFO] Ambari Server ..................................... SUCCESS > [14:19.226s] > [INFO] Ambari Agent ...................................... SUCCESS [15.514s] > [INFO] Ambari Client ..................................... SUCCESS [0.529s] > [INFO] > ------------------------------------------------------------------------ > > > Thanks, > > Sid Wagle > >
