https://bz.apache.org/bugzilla/show_bug.cgi?id=69552

--- Comment #8 from Mark Thomas <ma...@apache.org> ---
Is this addressing the correct problem?

That patch suggests that the issue is multiple calls to getBytes() for the same
MessageBytes instance that is currently set to the String type. Hence repeated
String to Bytes conversion causing the delay.

We already have a mechanism to cache the bytes - they are saved in the
ByteChunk. The subsequent calls should be made to getByteChunk().getBytes() /
getStart() / getLength()

Alternatively, we could add getBytesToType() method that would convert to bytes
and then change the type.

The complicating factor here is that the caller is responsible for keeping
track of which cached conversions are valid and which ones are out of date.

I think we need a better understanding of the callers to getBytes() to decide
on the best solution.

-- 
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to