Actually I am not sure there is a problem (I am just asking
and trying to locate the malfunction I came upon).
The one question I still don't know the answer:
Where the tool is returned back to the pool?
----- Original Message -----
From: "John McNally" <[EMAIL PROTECTED]>
To: "Turbine Users List" <[EMAIL PROTECTED]>
Sent: Wednesday, November 28, 2001 6:32 PM
Subject: Re: TurbinePullService + UIManager synchornization problem (Maybe
BUG) ?
> Once the object is given to a particular request from the pool, that
> object will not be given out again until it is returned. Where exactly
> do you see the problem? I don't see how you can tell that there is a
> problem with the pool code from the code given below.
>
> john mcnally
>
>
> Costas Stergiou wrote:
> >
> > iif you look at the code that populates the context with the request
> > scope tools, it just pools an object from the Pool (doesn't create
> > a new one each time). So, the same instance can be used for more than
> > one http requests.
> >
> > Here is the code from the TurbinePullService.java:
> >
> > /**
> > * Populate the given context with the request-scope tools
> > *
> > * @param context a Velocity Context to populate
> > * @param data a RunData instance
> > */
> > private void populateWithRequestTools(Context context, RunData data)
> > {
> > // Get the PoolService to fetch object instances from
> > PoolService pool = (PoolService)
> >
> > TurbineServices.getInstance().getService(PoolService.SERVICE_NAME);
> >
> > // Iterate the tools
> > Iterator it = requestTools.iterator();
> > while (it.hasNext())
> > {
> > ToolData toolData = (ToolData)it.next();
> > try
> > {
> > Object tool = pool.getInstance(toolData.toolClass);
> > if (tool instanceof ApplicationTool)
> > {
> > // request tools are init'd with a RunData object
> > ((ApplicationTool)tool).init(data);
> > }
> > // put the tool in the context
> > context.put(toolData.toolName, tool);
> > }
> > catch (Exception e)
> > {
> > Log.error(
> > "Could not instantiate tool " +
> > toolData.toolClassName
> > + " to add to the context",e);
> > }
> > }
> > }
> >
> > The only think that it does is that it init's the tool (with the
appropriate
> > rundata) before
> > adding it to the context. But actually it can be the same object
instance
> > that someone else
> > is using at the same moment!
> >
> > Is that so?
> > That's what I understand from the code above,
> > Please, correct me if I am wrong.
> > Thanks,
> > Costas
> >
> > ----- Original Message -----
> > From: "Eric Dobbs" <[EMAIL PROTECTED]>
> > To: "Turbine Users List" <[EMAIL PROTECTED]>
> > Sent: Wednesday, November 28, 2001 5:51 PM
> > Subject: Re: TurbinePullService + UIManager synchornization problem
(Maybe
> > BUG) ?
> >
> > >
> > > On Tuesday, November 27, 2001, at 12:39 AM, Costas Stergiou wrote:
> > >
> > > > Actually, when I mean "thread safe" I didn't mean "Java thread"
> > > > I meant "Request Safe" (actually it is thread safe also) but I don't
> > > > want to synchronize on the request tool of course.
> > > > But on a per-request basis, it is possible that the same instance of
the
> > > > tool is being used at the same time for more than one requests
> > > > (threads);
> > > > since a tool is only being taken from the Pool service and init'ed
but
> > > > there is no check that this tool is not being used by another
thread.
> > >
> > > Here's how I understand it. Someone correct me if I get this wrong.
> > >
> > > If the UIManager is configured to be used in the request scope, then
> > > a new UIManager object will be instantiated with each request (or
> > > pulled from the pool). So you get one UIManager object per request,
> > > and that should fix your problem.
> > >
> > > If it is configured for any other scope (which it is by default) then
> > > multiple requests (and therefore multiple threads) will be accessing
> > > the same instance of the UIManager. In this case, the UIManager needs
> > > to be modified for thread-safety.
> > >
> > > -Eric
> > >
> > > --
> > > To unsubscribe, e-mail:
> > <mailto:[EMAIL PROTECTED]>
> > > For additional commands, e-mail:
> > <mailto:[EMAIL PROTECTED]>
> > >
> >
> > --
> > To unsubscribe, e-mail:
<mailto:[EMAIL PROTECTED]>
> > For additional commands, e-mail:
<mailto:[EMAIL PROTECTED]>
>
> --
> To unsubscribe, e-mail:
<mailto:[EMAIL PROTECTED]>
> For additional commands, e-mail:
<mailto:[EMAIL PROTECTED]>
>
--
To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
