Re: 7.1 beta 3 Linux ODBC BEGIN Behaviour

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: "Hiroshi Inoue" <Inoue(at)tpf(dot)co(dot)jp>
Cc: "Hiroki Kataoka" <kataoka(at)interwiz(dot)koganei(dot)tokyo(dot)jp>, pgsql-interfaces(at)postgresql(dot)org, pgsql-odbc(at)postgresql(dot)org, "Steve Wranovsky" <stevew(at)merge(dot)com>
Subject: Re: 7.1 beta 3 Linux ODBC BEGIN Behaviour
Date: 2001-02-10 17:30:45
Message-ID: 3976.981826245@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-interfaces pgsql-odbc

"Hiroshi Inoue" <Inoue(at)tpf(dot)co(dot)jp> writes:
>> There must be "SELECT ~ FOR UPDATE" of inside of the transaction.

> You are right.
> However psqlodbc has never checked "for update".
> My recent change doesn't take "for update" into
> account either.

It'd be nice if ODBC could distinguish SELECT FOR UPDATE from plain
SELECT, but in practice it cannot reliably do so. Doubtless we could
extend ODBC to look for "FOR UPDATE" in the text of the query, but
that will only catch simple situations. Consider these possibilities:

* A view or rule invoked by the query uses FOR UPDATE. (Pre-7.1, we
didn't support FOR UPDATE in views ... but we do now.)

* A function invoked by the query does SELECT FOR UPDATE internally.

For that matter, it's quite possible for a function invoked by a SELECT
to do INSERT/UPDATE/DELETE internally. Therefore, it's impossible for
the ODBC driver to reliably distinguish a pure SELECT from a SELECT that
causes locking or even data updates.

Given these considerations, I think it's a mistake for ODBC to treat
SELECT differently from other queries for the purpose of setting
transaction boundaries.

regards, tom lane

In response to

Responses

Browse pgsql-interfaces by date

  From Date Subject
Next Message László Tibor 2001-02-10 18:15:56 RE: [INTERFACES] 7.1 beta 3 Linux ODBC BEGIN Behaviour
Previous Message Hiroshi Inoue 2001-02-10 14:22:19 RE: 7.1 beta 3 Linux ODBC BEGIN Behaviour

Browse pgsql-odbc by date

  From Date Subject
Next Message Peter Eisentraut 2001-02-10 18:09:23 Re: Re: [PATCHES] Fix for ODBC close
Previous Message Bruce Momjian 2001-02-10 17:17:13 Re: Re: [PATCHES] Fix for ODBC close