falcon-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Paul Isaychuk" <pisayc...@hortonworks.com>
Subject Re: Review Request 26639: [FALCON-801] Increase time to wait instances to got running
Date Wed, 15 Oct 2014 06:01:08 GMT


> On Oct. 14, 2014, 9:21 p.m., Raghav Gautam wrote:
> > falcon-regression/merlin/src/test/java/org/apache/falcon/regression/prism/OptionalInputTest.java,
line 281
> > <https://reviews.apache.org/r/26639/diff/1/?file=719373#file719373line281>
> >
> >     I think we are using 3 mins as of now - which is quite a lot of time. Why is
that not sufficient ?

In this test case instance which created after update have start time in a future. Thus they
wait not only for input data as usually but they wait on actual time when instance should
start. Thats why instances are created, got PREP state and then wait some time for their actual
start time.


- Paul


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


On Oct. 13, 2014, 10:29 a.m., Paul Isaychuk wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/26639/
> -----------------------------------------------------------
> 
> (Updated Oct. 13, 2014, 10:29 a.m.)
> 
> 
> Review request for Falcon.
> 
> 
> Bugs: FALCON-801
>     https://issues.apache.org/jira/browse/FALCON-801
> 
> 
> Repository: falcon-git
> 
> 
> Description
> -------
> 
> optionalTest_updateProcessMakeOptionalCompulsory needs more time for new instances to
start. They have got created and are on PREP state until they reach their start time.
> 
> 
> Diffs
> -----
> 
>   falcon-regression/merlin/src/test/java/org/apache/falcon/regression/prism/OptionalInputTest.java
c8a4754 
> 
> Diff: https://reviews.apache.org/r/26639/diff/
> 
> 
> Testing
> -------
> 
> test passed
> 
> 
> Thanks,
> 
> Paul Isaychuk
> 
>


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