ambari-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "jun aoki" <jun.aoki....@gmail.com>
Subject Re: Review Request 26510: AMBARI-7622 TestActionScheduler fails occasionally on builds.a.o stating expected:<ABORTED> but was:<PENDING>
Date Thu, 09 Oct 2014 22:38:46 GMT


> On Oct. 9, 2014, 10:28 p.m., Sid Wagle wrote:
> > ambari-server/src/test/java/org/apache/ambari/server/actionmanager/TestActionScheduler.java,
line 431
> > <https://reviews.apache.org/r/26510/diff/2/?file=717035#file717035line431>
> >
> >     The NPE seems to be from,
> >     line 782: commandParamsStageCache.put(stagePk, commandParams)
> >     
> >     Seems like we have command params as null so guava cache complains about it.
Maybe, "StageUtils.getGson().fromJson(s.getCommandParamsStage(), type)" returns null for empty
string, note: we are passing empty string for comandParamsStage.
> >     
> >     Not sure how this would be resolved after some time?
> >     A null check in ActionScheduler might not be a bad idea, your call.

Sid, yes it is from StageUtils.getGso()... call returning a null.
I'm not too sure either how it is resolved but it has been working this way and we solved
asychronous behaviour. I'd like to say it is good to go. Please give a Ship it if you are
OK.


- jun


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


On Oct. 9, 2014, 10:05 p.m., jun aoki wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/26510/
> -----------------------------------------------------------
> 
> (Updated Oct. 9, 2014, 10:05 p.m.)
> 
> 
> Review request for Ambari and Yusaku Sako.
> 
> 
> Bugs: AMBARI-7622
>     https://issues.apache.org/jira/browse/AMBARI-7622
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Tweaked the waiting condition upon ActionScheduler
> 
> 
> Diffs
> -----
> 
>   ambari-server/src/test/java/org/apache/ambari/server/actionmanager/TestActionScheduler.java
a20f252 
> 
> Diff: https://reviews.apache.org/r/26510/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> jun aoki
> 
>


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