RE: [jbehave-user] Bug in StoryManager regarding timeouts?
We have also faced this issue before. I think is a bug on jbehave, can you
please confirm?
To get around this we just set a super high timeout of 1 hour so the whole
story file has time to run
Thanks,
Enrique
From: Andreas Ebbert-Karroum [mailto:andreas.ebbert-karr...@codecentric.de]
Sent: Monday, July 15, 2013 7:37 AM
To: user@jbehave.codehaus.org
Subject: [jbehave-user] Bug in StoryManager regarding timeouts?
Hi,
as I read the specification of a timeout, it is setting the controls to let a
story time out, when a specified amount of time has passed. By the name this is
a setting per story. So, when I have a timeout of 300s per story, I should be
able to have 100 stories, which take 250 seconds each.
The way it is implemented is different, though. The StoryManager measures the
start time right after it kicked off all stories, and is waiting for all of
them to be completed.
https://github.com/jbehave/jbehave-core/blob/master/jbehave-core/src/main/java/org/jbehave/core/embedder/StoryManager.java
public void waitUntilAllDoneOrFailed(BatchFailures failures) {
>>> long start = System.currentTimeMillis();
boolean allDone = false;
while (!allDone) {
allDone = true;
for (RunningStory runningStory : runningStories.values()) {
Future future = runningStory.getFuture();
if (!future.isDone()) {
allDone = false;
>>> long durationInSecs = storyDurationInSecs(start);
long timeoutInSecs = embedderControls.storyTimeoutInSecs();
>>> if (durationInSecs > timeoutInSecs) {
Story story = runningStory.getStory();
StoryDuration storyDuration = new
StoryDuration(durationInSecs, timeoutInSecs);
embedderMonitor.storyTimeout(story, storyDuration);
storyRunner.cancelStory(story, storyDuration);
future.cancel(true);
}
break;
Is it just my thinking, or is that a bug in JBehave?
--
Mit freundlichen Grüßen / Best regards
Andreas Ebbert-Karroum | Agile Principal Consultant
codecentric AG | Merscheider Straße 1 | 42699 Solingen | Deutschland
tel: +49 (0) 212.23362825 | fax: +49 (0) 212.23362879 | mobil: +49 (0)
175.2664109
www.codecentric.de |
blog.codecentric.de |
www.meettheexperts.de |
www.more4fi.de
Sitz der Gesellschaft: Düsseldorf | HRB 63043
Vorstand: Klaus Jäger (Vorsitzender) . Michael Hochgürtel . Mirko Novakovic .
Rainer Vehns
Aufsichtsrat: Patric Fedlmeier (Vorsitzender) . Bernd Klinkmann . Jürgen Schütz
Diese E-Mail einschließlich evtl. beigefügter Dateien enthält vertrauliche
und/oder rechtlich geschützte Informationen. Wenn Sie nicht der richtige
Adressat sind oder diese E-Mail irrtümlich erhalten haben, informieren Sie
bitte sofort den Absender und löschen Sie diese E-Mail und evtl. beigefügter
Dateien umgehend. Das unerlaubte Kopieren, Nutzen oder Öffnen evtl. beigefügter
Dateien sowie die unbefugte Weitergabe dieser E-Mail ist nicht gestattet.
Re: [jbehave-user] Bug in StoryManager regarding timeouts?
Hi Andreas,
yes, currently the timeout is a global one. It could be enhanced to be
story specific. Please raise a JIRA issue for it.
Cheers
On 15/07/2013 15:36, Andreas Ebbert-Karroum wrote:
Hi,
as I read the specification of a timeout, it is setting the controls
to let a story time out, when a specified amount of time has passed.
By the name this is a setting per story. So, when I have a timeout of
300s per story, I should be able to have 100 stories, which take 250
seconds each.
The way it is implemented is different, though. The StoryManager
measures the start time right after it kicked off all stories, and is
waiting for all of them to be completed.
https://github.com/jbehave/jbehave-core/blob/master/jbehave-core/src/main/java/org/jbehave/core/embedder/StoryManager.java
public void waitUntilAllDoneOrFailed(BatchFailures failures) { >>>
long start = System.currentTimeMillis(); boolean allDone = false;
while (!allDone) { allDone = true; for (RunningStory runningStory
: runningStories.values()) { Future future =
runningStory.getFuture(); if (!future.isDone()) { allDone = false;
>>> long durationInSecs = storyDurationInSecs(start); long
timeoutInSecs = embedderControls.storyTimeoutInSecs(); >>> if
(durationInSecs > timeoutInSecs) { Story story =
runningStory.getStory(); StoryDuration storyDuration = new
StoryDuration(durationInSecs, timeoutInSecs);
embedderMonitor.storyTimeout(story, storyDuration);
storyRunner.cancelStory(story, storyDuration);
future.cancel(true); } break;
Is it just my thinking, or is that a bug in JBehave?
--
Mit freundlichen Grüßen / Best regards
Andreas Ebbert-Karroum | Agile Principal Consultant
codecentric AG | Merscheider Straße 1 | 42699 Solingen | Deutschland
tel: +49 (0) 212.23362825 | fax: +49 (0) 212.23362879 | mobil: +49 (0)
175.2664109
www.codecentric.de | blog.codecentric.de
| www.meettheexperts.de
| www.more4fi.de
Sitz der Gesellschaft: Düsseldorf | HRB 63043
Vorstand: Klaus Jäger (Vorsitzender) . Michael Hochgürtel . Mirko
Novakovic . Rainer Vehns
Aufsichtsrat: Patric Fedlmeier (Vorsitzender) . Bernd Klinkmann .
Jürgen Schütz
Diese E-Mail einschließlich evtl. beigefügter Dateien enthält
vertrauliche und/oder rechtlich geschützte Informationen. Wenn Sie
nicht der richtige Adressat sind oder diese E-Mail irrtümlich erhalten
haben, informieren Sie bitte sofort den Absender und löschen Sie diese
E-Mail und evtl. beigefügter Dateien umgehend. Das unerlaubte
Kopieren, Nutzen oder Öffnen evtl. beigefügter Dateien sowie die
unbefugte Weitergabe dieser E-Mail ist nicht gestattet.
Re: [jbehave-user] Bug in StoryManager regarding timeouts?
Done:
http://jira.codehaus.org/browse/JBEHAVE-933
2013/7/16 Mauro Talevi
> Hi Andreas,
>
> yes, currently the timeout is a global one. It could be enhanced to be
> story specific. Please raise a JIRA issue for it.
>
> Cheers
>
>
> On 15/07/2013 15:36, Andreas Ebbert-Karroum wrote:
>
> Hi,
>
> as I read the specification of a timeout, it is setting the controls to
> let a story time out, when a specified amount of time has passed. By the
> name this is a setting per story. So, when I have a timeout of 300s per
> story, I should be able to have 100 stories, which take 250 seconds each.
>
> The way it is implemented is different, though. The StoryManager
> measures the start time right after it kicked off all stories, and is
> waiting for all of them to be completed.
>
>
> https://github.com/jbehave/jbehave-core/blob/master/jbehave-core/src/main/java/org/jbehave/core/embedder/StoryManager.java
>
>
>> public void waitUntilAllDoneOrFailed(BatchFailures failures) {
>>
>> >>> long start = System.currentTimeMillis();
>>
>> boolean allDone = false;
>>
>> while (!allDone) {
>>
>> allDone = true;for (RunningStory runningStory :
>> runningStories.values()) {
>>
>> Future future = runningStory.getFuture();
>>
>> if (!future.isDone()) {
>>
>> allDone = false;>>> long durationInSecs
>> = storyDurationInSecs(start);
>>
>> long timeoutInSecs =
>> embedderControls.storyTimeoutInSecs();
>>
>> >>> if (durationInSecs > timeoutInSecs) {
>>
>> Story story = runningStory.getStory();
>>
>> StoryDuration storyDuration = new
>> StoryDuration(durationInSecs, timeoutInSecs);
>>
>> embedderMonitor.storyTimeout(story, storyDuration);
>>
>> storyRunner.cancelStory(story, storyDuration);
>>
>> future.cancel(true);
>>
>> }break;
>
>
> Is it just my thinking, or is that a bug in JBehave?
>
>
>
>
> --
> Mit freundlichen Grüßen / Best regards
>
> Andreas Ebbert-Karroum | Agile Principal Consultant
>
> codecentric AG | Merscheider Straße 1 | 42699 Solingen | Deutschland
> tel: +49 (0) 212.23362825 | fax: +49 (0) 212.23362879 | mobil: +49 (0)
> 175.2664109
> www.codecentric.de | blog.codecentric.de | www.meettheexperts.de |
> www.more4fi.de
>
> Sitz der Gesellschaft: Düsseldorf | HRB 63043
> Vorstand: Klaus Jäger (Vorsitzender) . Michael Hochgürtel . Mirko
> Novakovic . Rainer Vehns
> Aufsichtsrat: Patric Fedlmeier (Vorsitzender) . Bernd Klinkmann . Jürgen
> Schütz
>
> Diese E-Mail einschließlich evtl. beigefügter Dateien enthält vertrauliche
> und/oder rechtlich geschützte Informationen. Wenn Sie nicht der richtige
> Adressat sind oder diese E-Mail irrtümlich erhalten haben, informieren Sie
> bitte sofort den Absender und löschen Sie diese E-Mail und evtl.
> beigefügter Dateien umgehend. Das unerlaubte Kopieren, Nutzen oder Öffnen
> evtl. beigefügter Dateien sowie die unbefugte Weitergabe dieser E-Mail ist
> nicht gestattet.
>
>
>
--
Mit freundlichen Grüßen / Best regards
Andreas Ebbert-Karroum | Agile Principal Consultant
codecentric AG | Merscheider Straße 1 | 42699 Solingen | Deutschland
tel: +49 (0) 212.23362825 | fax: +49 (0) 212.23362879 | mobil: +49 (0)
175.2664109
www.codecentric.de | blog.codecentric.de | www.meettheexperts.de |
www.more4fi.de
Sitz der Gesellschaft: Düsseldorf | HRB 63043
Vorstand: Klaus Jäger (Vorsitzender) . Michael Hochgürtel . Mirko Novakovic
. Rainer Vehns
Aufsichtsrat: Patric Fedlmeier (Vorsitzender) . Bernd Klinkmann . Jürgen
Schütz
Diese E-Mail einschließlich evtl. beigefügter Dateien enthält vertrauliche
und/oder rechtlich geschützte Informationen. Wenn Sie nicht der richtige
Adressat sind oder diese E-Mail irrtümlich erhalten haben, informieren Sie
bitte sofort den Absender und löschen Sie diese E-Mail und evtl.
beigefügter Dateien umgehend. Das unerlaubte Kopieren, Nutzen oder Öffnen
evtl. beigefügter Dateien sowie die unbefugte Weitergabe dieser E-Mail ist
nicht gestattet.
