Re: BUG #15977: Inconsistent behavior in chained transactions

From: fn ln <emuser20140816(at)gmail(dot)com>
To: Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com>
Cc: Fabien COELHO <coelho(at)cri(dot)ensmp(dot)fr>, Michael Paquier <michael(at)paquier(dot)xyz>, Andres Freund <andres(at)anarazel(dot)de>, PostgreSQL Developers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: BUG #15977: Inconsistent behavior in chained transactions
Date: 2019-09-09 03:58:46
Message-ID: CA+99BHqxMfUyZcSh2B=ni5db83U1qtgX7bv+U7iXRbJqkTVFJg@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs pgsql-hackers

Confirmed. Thank you all for your help.

The only concern is that this test:

SET TRANSACTION READ WRITE\; COMMIT AND CHAIN; -- error
SHOW transaction_read_only;

SET TRANSACTION READ WRITE\; ROLLBACK AND CHAIN; -- error
SHOW transaction_read_only;

makes more sense with READ ONLY because default_transaction_read_only is
off at this point.

2019年9月9日(月) 5:27 Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com>:

> On 2019-09-07 18:32, fn ln wrote:
> >> Missed them somehow. But I don't think they're quite sufficient. I think
> >> at least we also need tests that test things like multi-statement
> >> exec_simple-query() *with* explicit transactions and chaining.
> > Added a few more tests for that.
>
> I committed this patch with some cosmetic changes and documentation
> updates.
>
> --
> Peter Eisentraut http://www.2ndQuadrant.com/
> PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
>

In response to

Responses

Browse pgsql-bugs by date

  From Date Subject
Next Message Tom Lane 2019-09-09 04:22:38 Re: BUG #15995: VACUUM not working after setting/unsetting of archive_mode = on, wal_level = replica
Previous Message PG Bug reporting form 2019-09-09 02:12:22 BUG #15995: VACUUM not working after setting/unsetting of archive_mode = on, wal_level = replica

Browse pgsql-hackers by date

  From Date Subject
Next Message Michael Paquier 2019-09-09 05:28:14 Re: refactoring - share str2*int64 functions
Previous Message Amit Kapila 2019-09-09 03:05:39 Re: [HACKERS] [PATCH] pageinspect function to decode infomasks