Carsten Ziegeler wrote:
> Might be worth investigating into running modes. Hmm.
BTW, I'm happy this discussion came to nice indication and real
solutions, kudos to all that partecipate :-)
> But comming back to the original problem ;) - We already have an
> intermediate
> output stream implemented and noone really complained about it:
> Its the buffered output stream in the abstract text serializer...the only
> problem here is that a) the size of the buffer might be too small
> and b) the stream is flushed on recycle().
>
> So, if we
> a) Move this buffered output stream handling out of the serializer
> into the environment
+1 There is where it belongs architecturally IMHO, thanks for pointing
this out, I wasn't aware.
> b) Don't flush the stream on recycling a serializer
Even better: prevent flushng alltogether by making a
CocoonIntermediateStream that extends the output stream, and that does
nothing on flush, and instead uses an Environment-protected method for
flushing, so we can play safe.
> c) Add a small logic checking the buffer size for the current response,
+1 and make it configurable.
> we have everything we need.
>
> This should work and it should also don't have any impact on the
> performance as 99% of it is already done.
At least we can see how it works.
Decoupling from the real stream gives us nice advantages it seems.
> I will give this a try today - and we can see if it works.
:-)
> But we should of course continue this discussion about the running modes.
Running modes is a nice idea that we already had sometime back, and that
now fortunately came back.
Basically with a single param it should be able to switch configurations.
Hence we can switch from a programmer env conf to a production one quite
easily.
--
Nicola Ken Barozzi [EMAIL PROTECTED]
- verba volant, scripta manent -
(discussions get forgotten, just code remains)
---------------------------------------------------------------------
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, email: [EMAIL PROTECTED]
