| From: | Anders Bengtsson <ndrsbngtssn(at)yahoo(dot)se> |
|---|---|
| To: | Barry Lind <barry(at)xythos(dot)com> |
| Cc: | <pgsql-jdbc(at)postgresql(dot)org>, <pgsql-patches(at)postgresql(dot)org> |
| Subject: | Re: [PATCH] Cleanup of JDBC character encoding |
| Date: | 2001-07-10 10:35:26 |
| Message-ID: | Pine.LNX.4.33.0107101226570.4394-100000@sensenet.localdomain |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-jdbc pgsql-patches |
On Mon, 9 Jul 2001, Barry Lind wrote:
> Anders,
>
> I have some issues with this patch.
>
> 1) Please don't require the driver to perform yet another round trip to
> the server to get the database encoding. You changed the following code
> which accessed the server once at connection creation:
[snip]
> 2) I don't like having test specific code in Encoding.java, specifically
> the dummy encoding 'X-UNIT-TESTS'.
Thanks Barry, I'll fix these things and return with a new patch.
I think I'll be able to get pretty good test coverage even without the
extra test data.
/Anders
_____________________________________________________________________
A n d e r s B e n g t s s o n ndrsbngtssn(at)yahoo(dot)se
Stockholm, Sweden
_________________________________________________________
Do You Yahoo!?
Get your free @yahoo.com address at http://mail.yahoo.com
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Anders Bengtsson | 2001-07-10 12:31:43 | Re: [PATCHES] Re: [PATCH] Cleanup of JDBC character encoding |
| Previous Message | Thomas Lockhart | 2001-07-10 02:27:06 | Re: Re: Backups WAS: 2 gig file size limit |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Anders Bengtsson | 2001-07-10 12:31:43 | Re: [PATCHES] Re: [PATCH] Cleanup of JDBC character encoding |
| Previous Message | Thomas Lockhart | 2001-07-10 02:27:06 | Re: Re: Backups WAS: 2 gig file size limit |