From: Gunnar R|nning <gunnar@candleweb.no>
To: Barry Lind <barry@xythos.com>
Cc: Peter Mount <peter@retep.org.uk>
Cc: pgsql-interfaces@postgresql.org
Subject: Re: [PATCHES] Re: Fixes and enhancements to JDBC driver (take 2)
Date: 19 Jan 2001 01:53:57 +0100
Message-ID: <x68zo8qs4a.fsf@thor.candleweb.no> (raw)
References: <Pine.LNX.4.10.10101171425190.26916-200000@polymorphic.microstate.com>
<x6bst5dl7g.fsf@thor.candleweb.no>
<3A676B76.77944038@xythos.com>
Barry Lind <barry@xythos.com> writes:
> Gunner,
>
> Do your fixes address the concerns I raised on 12/21? (I have included
> that email to the list below). To summarize the three major
> concerns/bugs were:
> 1) Code incorrectly deallocates when a new statement is executed, even
> though the byte[]s are still being used.
> 2) The cache isn't limited in size, resulting in essentially memory
> leaks for long lived connections in a connection pool.
> 3) The implementation is limited to a max 256 byte byte[], whereas my
> queries have many values that exceed this size, and the current
> implementation doesn't lend itself well (because of #2) to cache things
> upto 8K in size.
The original patch that I supplied was a proof of concept on what kind of
performance improvements that could be made by reusing byte arrays. This
was unfortunately committed before anybody but me had done any testing at
all on it.
The most serious problem with this code was your issue 1). Number 2) and 3)
should be easy to handle has config parameters. The reason for hardcoding
3) to 256 was simply because I found this to be the most optimal value for
the web application I was doing the testing on.
Eventually, it should be configurable whether to use the byte[] caching
implementation or not, as the perfomance of memory allocation may vary
greatly depending on VM and OS implementations.
If you go back to the October archives of pgsql-general you will find a
pointer to my second shot at an implementation - this one fix your issue 1)
but not the others.
I would like to see what you have been working on as well, so we can come
up with the best of breeds solution.
Regards,
Gunnar
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: pgsql-interfaces@postgresql.org
Cc: gunnar@candleweb.no, barry@xythos.com, peter@retep.org.uk
Subject: Re: [PATCHES] Re: Fixes and enhancements to JDBC driver (take 2)
In-Reply-To: <x68zo8qs4a.fsf@thor.candleweb.no>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox