[Fwd: Re: Fiber suspend and resume behavior]

[Potociar Marek]

Hello Jitu,
could you please have a look at this?
Thanks,
Marek

-------- Original Message --------
Subject: Re: Fiber suspend and resume behavior
Date: Tue, 22 Jan 2008 15:58:43 +0100
From: Marek Potociar <Marek.Potociar@...>
Reply-To: dev@...
To: dev@...
CC: dev@...
References: <4796022B.6030101@...>



Btw., could this 1-line patch possibly solve the problem?

Marek

Marek Potociar wrote:

> Hello,
> I wonder if the existing implemented logic of suspending and resuming
> a fiber is correct. Intuitively, I would expect that a fiber suspended
> during a request processing would, after resuming, continue with a
> call to next.processRequest(...)[1], which is not the currently
> implement behavior (right now the resumed fiber ignores next tube and
> immediately calls processResponse() on the last tube in the stack -
> the one that suspended the fiber). IOW, currently the request
> processing cannot be really suspended as resume is available only for
> response processing.
>
> I would like to ask if you could change the existing implementation to
> allow the "natural" suspend behavior. Shall I file a RFE?
>
> Thank you,
> Marek
>
> [1] I have a use case where I would like to use this suspend behavior.
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@...
> For additional commands, e-mail: dev-help@...
>

# This patch file was generated by NetBeans IDE
# This patch can be applied using context Tools: Apply Diff Patch action on respective folder.
# It uses platform neutral UTF-8 encoding.
# Above lines and this line are ignored by the patching process.
Index: jax-ws-sources/jaxws-ri/rt/src/com/sun/xml/ws/api/pipe/Fiber.java
--- jax-ws-sources/jaxws-ri/rt/src/com/sun/xml/ws/api/pipe/Fiber.java Base (1.3)
+++ jax-ws-sources/jaxws-ri/rt/src/com/sun/xml/ws/api/pipe/Fiber.java Locally Modified (Based On 1.3)
@@ -629,7 +629,7 @@
                         break;
                     case NextAction.SUSPEND:
                         pushCont(last);
-                        next = null;
+                        next = na.next;
                         suspend();
                         break;
                     default:



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@...
For additional commands, e-mail: dev-help@...



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@...
For additional commands, e-mail: dev-help@...