Re: Query ResultSet parsing speedup patch (resend)

On Sat, 30 Sep 2006, Mikko Tiihonen wrote:

> This is a resend since I did not get any feedback to this patch when I
> sent it two months ago. Could someone at least have some recommendations
> what I should do to get this patch evaluated or even applied to the
> official source.
>

I like the change to use VisibleBufferedInputStream.  That's a clean way
of avoiding copies.  I don't particularly like the number parsing changes
as they're a pretty ugly hack, but I can't deny the performance
improvements (I saw 30-35%).  I plan on applying this unless someone
objects.

One bug I found was that you have the wrong bytes.length check in
getFastInt.  It can handle one more byte than you specified.

Kris Jurka