ambari-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "John Speidel" <jspei...@hortonworks.com>
Subject Re: Review Request 20346: Stack definition does not provide global properties with empty values
Date Tue, 15 Apr 2014 18:30:30 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20346/#review40418
-----------------------------------------------------------


There are still many props with empty values in stack definition which haven't been updated.
For example, 2.0.5/services/HBASE/configuration/hbase-site.xml/hbase.tmp.dir.  This property
was just randomly picked from the list.

Are there no valid defaults for these properties?


ambari-server/src/main/java/org/apache/ambari/server/api/util/StackExtensionHelper.java
<https://reviews.apache.org/r/20346/#comment73405>

    Why would the property name be null or empty?  Was there an issue which prompted adding
this check?



ambari-server/src/main/java/org/apache/ambari/server/api/util/StackExtensionHelper.java
<https://reviews.apache.org/r/20346/#comment73407>

    It is not clear to me what this log msg means.



ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HBASE/configuration/global.xml
<https://reviews.apache.org/r/20346/#comment73415>

    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 ...
    
    


- John Speidel


On April 15, 2014, 1:09 a.m., Sid Wagle wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/20346/
> -----------------------------------------------------------
> 
> (Updated April 15, 2014, 1:09 a.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-web/app/data/HDP2/site_properties.js dedd4ca 
>   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
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message