Re: JDBC: One more autocommit use to work around

Fernando,

Thanks for the catch.  A fix will be checked in shortly.

thanks,
--Barry


Fernando Nasser wrote:
> Last night's patch to remove uses of autocommit missed this spot.
>
> Note that here one cannot use the same strategy of ignoring the
> existence of autocommit for 7.3 as it was done on the other cases as
> someone may have configured their backend to start in SQL standard mode
> (i.e., autocommit off).
>
>
> ------------------------------------------------------------------------
>
> Index: org/postgresql/jdbc1/AbstractJdbc1Connection.java
> ===================================================================
> RCS file: /projects/cvsroot/pgsql-server/src/interfaces/jdbc/org/postgresql/jdbc1/AbstractJdbc1Connection.java,v
> retrieving revision 1.19
> diff -c -p -r1.19 AbstractJdbc1Connection.java
> *** org/postgresql/jdbc1/AbstractJdbc1Connection.java    29 May 2003 03:21:32 -0000    1.19
> --- org/postgresql/jdbc1/AbstractJdbc1Connection.java    29 May 2003 13:54:02 -0000
> *************** public abstract class AbstractJdbc1Conne
> *** 778,784 ****
>           //We also set the client encoding so that the driver only needs
>           //to deal with utf8.  We can only do this in 7.3 because multibyte
>           //support is now always included
> !         if (haveMinimumServerVersion("7.3"))
>           {
>               BaseResultSet acRset =
>                   execSQL("set client_encoding = 'UNICODE'; show autocommit");
> --- 778,784 ----
>           //We also set the client encoding so that the driver only needs
>           //to deal with utf8.  We can only do this in 7.3 because multibyte
>           //support is now always included
> !         if (haveMinimumServerVersion("7.3") && !haveMinimumServerVersion("7.4"))
>           {
>               BaseResultSet acRset =
>                   execSQL("set client_encoding = 'UNICODE'; show autocommit");
> *************** public abstract class AbstractJdbc1Conne
> *** 798,803 ****
> --- 798,813 ----
>               {
>                   execSQL("set autocommit = on; commit;");
>               }
> +         }
> +         // On 7.4 there is no more backend autocommit, but we still need
> +         // to set the client encoding.
> +         else if (haveMinimumServerVersion("7.4"))
> +         {
> +             BaseResultSet acRset =
> +                 execSQL("set client_encoding = 'UNICODE'");
> +
> +             //set encoding to be unicode
> +             encoding = Encoding.getEncoding("UNICODE", null);
>           }
>
>           // Initialise object handling