From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Mark Dilger <hornschnorter(at)gmail(dot)com> |
Cc: | pgsql-hackers(at)lists(dot)postgresql(dot)org, Martijn van Oosterhout <kleptog(at)gmail(dot)com> |
Subject: | Re: LISTEN/NOTIFY testing woes |
Date: | 2019-11-24 19:04:38 |
Message-ID: | 21961.1574622278@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Mark Dilger <hornschnorter(at)gmail(dot)com> writes:
> On 11/24/19 10:39 AM, Tom Lane wrote:
>> After sleeping on it, I'm not really happy with what I did in
>> PrepareTransaction (that is, invent a separate PrePrepare_Notify
>> function). The idea was to keep that looking parallel to what
>> CommitTransaction does, and preserve infrastructure against the
>> day that somebody gets motivated to allow LISTEN or NOTIFY in
>> a prepared transaction. But on second thought, what would surely
>> happen when that feature gets added is just that AtPrepare_Notify
>> would serialize the pending LISTEN/NOTIFY actions into the 2PC
>> state file. There wouldn't be any need for PrePrepare_Notify,
>> so there's no point in introducing that. I'll just move the
>> comment saying that nothing has to happen at that point for NOTIFY.
> I assumed you had factored it out in anticipation of supporting notify
> here in the future. If you want to backtrack that decision and leave it
> inline, you would still keep the test rather than just a comment, right?
No, there wouldn't be any error condition; that's just needed because the
feature isn't implemented yet. So I'll leave that alone; the only thing
that needs to happen now in the PREPARE code path is to adjust the one
comment.
regards, tom lane
From | Date | Subject | |
---|---|---|---|
Next Message | Peter Geoghegan | 2019-11-24 19:07:37 | Re: [PATCH] Fix possible underflow in expression (maxoff - 1) |
Previous Message | Mark Dilger | 2019-11-24 19:01:04 | Re: LISTEN/NOTIFY testing woes |