Re: Re: [PATCH] Cleanup of JDBC character encoding - Mailing list pgsql-patches

From Anders Bengtsson
Subject Re: Re: [PATCH] Cleanup of JDBC character encoding
Date
Msg-id Pine.LNX.4.33.0107101424400.5824-400000@sensenet.localdomain
Whole thread Raw
In response to Re: [PATCH] Cleanup of JDBC character encoding  (Anders Bengtsson <ndrsbngtssn@yahoo.se>)
Responses Re: [JDBC] Re: Re: [PATCH] Cleanup of JDBC character encoding
List pgsql-patches
> > 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'.

Ok, now I've made the changes, and added a comment about why the code for
(1) should stay optimized. :)

/Anders
_____________________________________________________________________
A n d e r s  B e n g t s s o n                   ndrsbngtssn@yahoo.se
Stockholm, Sweden

Attachment

pgsql-patches by date:

Previous
From: Anders Bengtsson
Date:
Subject: Re: [PATCH] Cleanup of JDBC character encoding
Next
From: Jason Tishler
Date:
Subject: Re: Python (PyGreSQL) make install patch