I think the stage has to be maintained in the Request object.
No, definitely not. Here's how MinTC (MinimalTomcat) does
it (this is alpha code, and I've deleted some of the methods
to keep the size down):
I think I would have implemented it that way also.
The thing is that the current
Hi Remy,
I think the performance related change that you made to StandardPipeline can
be improved upon in that it can avoid using a HashMap to store/retrieve the
pipeline stage and instead simply store/retrieve it from an integer variable
in the RequestBase class. Since this codepath
Arvind Srinivasan wrote:
I think the performance related change that you made to StandardPipeline can
be improved upon in that it can avoid using a HashMap to store/retrieve the
pipeline stage and instead simply store/retrieve it from an integer variable
in the RequestBase class. Since this
Christopher St. John wrote:
The obvious implementation is to have have ValveContext hold
the index.
That's how I assumed it worked the first time I
saw the Pipeline, Valve and ValveContext classes. I was suprised
to see a ThreadLocal. Using ValveContext has the appropriate
threading
Arvind Srinivasan wrote:
Christopher St. John wrote:
The obvious implementation is to have have ValveContext hold
the index.
But isn't a ValveContext (Pipeline) shared across requests ?
It's shouldn't be, but it is now. StandardPipeline
implements VavleContext, but there's no
Christopher St. John wrote:
No, definitely not. Here's how MinTC (MinimalTomcat) does
it (this is alpha code, and I've deleted some of the methods
to keep the size down):
Thanks..I like MinTC's solution.
Arvind
--
To unsubscribe, e-mail: mailto:[EMAIL PROTECTED]
For additional