Hi,

On 9/20/07, Thomas Mueller <[EMAIL PROTECTED]> wrote:
> You have also commented out setAutoCommit(true):
>
> --- BundleDbPersistenceManager
>         con = DriverManager.getConnection(url, user, password);
> -        con.setAutoCommit(true);
> +        //con.setAutoCommit(true);

Hmm, that's correct. It's an accidental change in revision 535830 that
I just merged to the 1.3 branch. I'll merge also the fix from revision
535834.

> In my view this line can be completely removed, as by default
> autoCommit is true anyway (for each database).

It might be set otherwise in some local configuration or even the
database connection string, so I guess it would make sense to
explicitly set the autocommit status.

BR,

Jukka Zitting

Reply via email to