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 37294: HDFS check fails after move NameNode on NN HA cluster
Date Tue, 11 Aug 2015 17:56:50 GMT

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

Ship it!


Patch looks good.

I am not sure that I understand your comment regarding the removal of non-pending host requests
from the allHostRequests collection.  Doing this would be incorrect.  We still need access
to all host requests, even completed hosts requests.  One example of why we need access to
completed host requests is to obtain request/stage/command information.

- John Speidel


On Aug. 11, 2015, 5:16 p.m., Dmytro Sen wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/37294/
> -----------------------------------------------------------
> 
> (Updated Aug. 11, 2015, 5:16 p.m.)
> 
> 
> Review request for Ambari, John Speidel and Robert Nettleton.
> 
> 
> Bugs: AMBARI-12688
>     https://issues.apache.org/jira/browse/AMBARI-12688
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> This ticket is related to
> https://issues.apache.org/jira/browse/AMBARI-10750 Initial merge of advanced api provisioning
work
> Detailed scenario
> 1. Deploy 3-node cluster from a blueprint (NameNode is on host1. Here NN is mapped to
host1 in the cluster topology)
> 2. Enable NN HA (now NameNodes are on host1 and host2) 
> 3. Move NN from host1 to host3(now NameNodes are on host3 and host2, but according to
cluster topology NN also mapped to host1)
> 4. HDFS service check fails
> 
> 
> Diffs
> -----
> 
>   ambari-server/src/main/java/org/apache/ambari/server/topology/LogicalRequest.java b7f95cf

>   ambari-server/src/main/java/org/apache/ambari/server/topology/TopologyManager.java
8e7fb7f 
>   ambari-server/src/main/java/org/apache/ambari/server/utils/StageUtils.java 3da0fe2

>   ambari-server/src/test/java/org/apache/ambari/server/controller/KerberosHelperTest.java
5d84fbc 
>   ambari-server/src/test/java/org/apache/ambari/server/utils/StageUtilsTest.java 7cc70e8

> 
> Diff: https://reviews.apache.org/r/37294/diff/
> 
> 
> Testing
> -------
> 
> [INFO] Ambari Server ..................................... SUCCESS [1:14:15.296s]
> [INFO] ------------------------------------------------------------------------
> [INFO] BUILD SUCCESS
> 
> 
> Thanks,
> 
> Dmytro Sen
> 
>


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