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




contrib/views/utils/src/main/java/org/apache/ambari/view/utils/hdfs/ConfigurationBuilder.java
Lines 270 (patched)
<https://reviews.apache.org/r/58480/#comment246134>

    !StringUtils.isEmpty(namenodeIDs) should be enough.  That will check 
whether the string is null or empty.


- Tim Thorpe


On April 20, 2017, 8:53 p.m., Sangeeta Ravindran wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/58480/
> -----------------------------------------------------------
> 
> (Updated April 20, 2017, 8:53 p.m.)
> 
> 
> Review request for Ambari, Alexandr Antonenko, Di Li, DIPAYAN BHOWMICK, and 
> Tim Thorpe.
> 
> 
> Bugs: AMBARI-20354
>     https://issues.apache.org/jira/browse/AMBARI-20354
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> 1. In a multi-node cluster where namenode HA is not enabled, create a Files 
> View. Select Custom Configuration and enter the WebHDFS FileSystem URI*. In 
> the "List of NameNodes", enter the namenode information. Save and execute 
> view. There are no errors.
> 2. Edit the view and delete the values entered in the "List of NameNodes" 
> field. Save and execute the view.
> The following error is thrown although Namenode HA is not enabled.
> Service 'hdfs' check failed: HDFS080 webhdfs.ha.namenodes.list namenodes 
> count is not exactly 2
> 
> 
> Diffs
> -----
> 
>   
> contrib/views/utils/src/main/java/org/apache/ambari/view/utils/hdfs/ConfigurationBuilder.java
>  7e174a7 
> 
> 
> Diff: https://reviews.apache.org/r/58480/diff/1/
> 
> 
> Testing
> -------
> 
> Manual testing.
> 
> 
> Thanks,
> 
> Sangeeta Ravindran
> 
>

Reply via email to