ambari-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jaimin Jetly" <jai...@hortonworks.com>
Subject Re: Review Request 27547: UI: Change "dfs.namenode.safemode.threshold-pct" value if HA enabled
Date Tue, 04 Nov 2014 22:01:36 GMT

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



ambari-web/app/utils/config.js
<https://reviews.apache.org/r/27547/#comment101136>

    We might not want to introduce this function over here.
    
    So on enabling HA when we go to HDFS page, this config will be changed by the new code:
    1) An undo sign will be present without user having made any changes expicitly and navigating
out of this page will warn user to save the change before navigating.
    2) User will not be able to change this config value via UI. If user changes this config
then it will be saved successfully but after that again 0.99f will appear as the value of
the config forcing it to save the config with only this value. 
    
    Instead I suggest to update this config with 0.99f only once on NameNode HA wizard (Review
page that displays all properties that will change as part of this wizard)


- Jaimin Jetly


On Nov. 4, 2014, 9:05 p.m., Xi Wang wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/27547/
> -----------------------------------------------------------
> 
> (Updated Nov. 4, 2014, 9:05 p.m.)
> 
> 
> Review request for Ambari, Jaimin Jetly and Yusaku Sako.
> 
> 
> Bugs: Ambari-8129
>     https://issues.apache.org/jira/browse/Ambari-8129
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> hdfs-site.xml
> Change value of "dfs.namenode.safemode.threshold-pct" to "0.99f" if NN HA
> 
> 
> Diffs
> -----
> 
>   ambari-web/app/utils/config.js 2f157e8 
> 
> Diff: https://reviews.apache.org/r/27547/diff/
> 
> 
> Testing
> -------
> 
> Tested patch before/after NN HA enabled. Unit tests passed.
> 
> 
> Thanks,
> 
> Xi Wang
> 
>


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