Re: commit and rollback don't throw exceptions when they should

Kris Jurka <books@ejurka.com> wrote:

> I've commited a modified version of this patch to HEAD only,
> because changing this behavior could break peoples applications.
>
> I extended the check for the connection already being closed to the
> other connection methods.  There's no reason commit/rollback are
> special here.

That all makes sense.  I think you have a copy/paste error in the
exception message, though: "This statement has been closed."  It is
the connection which has been closed here.

Also, for software which tries to make sense of the SQLSTATE for
automated handling, I'm inclined to think that something in Class Code
'08' (Connection Exception) would be more useful than the Class Code
'55' (Object Not in Prerequisite State) value.  Granted, it's a gray
area, since the problem with the connection is that it's not in an
open state....

-Kevin