ambari-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Dmitro Lisnichenko" <dlysniche...@hortonworks.com>
Subject Re: Review Request 40139: [preview] SKIPPED_FAILED state should not be bubbled up to the Upgrade level
Date Tue, 10 Nov 2015 16:15:05 GMT

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



ambari-server/src/main/java/org/apache/ambari/server/controller/internal/CalculatedStatus.java
(line 366)
<https://reviews.apache.org/r/40139/#comment164586>

    If we remove this line, and any stage has failures, the whole request will be still marked
as FAILED.
    But if we consider skipped failures as nothing serious, then probably removal of this
line should do the trick


- Dmitro Lisnichenko


On Nov. 10, 2015, 6:12 p.m., Dmitro Lisnichenko wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/40139/
> -----------------------------------------------------------
> 
> (Updated Nov. 10, 2015, 6:12 p.m.)
> 
> 
> Review request for Ambari, Nate Cole and Yusaku Sako.
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Uploaded a patch for discussion
> 
> 
> Diffs
> -----
> 
>   ambari-server/src/main/java/org/apache/ambari/server/controller/internal/CalculatedStatus.java
f87c32c 
> 
> Diff: https://reviews.apache.org/r/40139/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Dmitro Lisnichenko
> 
>


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