Hi,
On 2019-05-02 16:54:11 -0400, Tom Lane wrote:
> I just finished a successful run of the core regression tests with CCA.
> Given the calendar, I think that's about as much CCA testing as I should
> do personally. I'll make a cleanup pass on this patch and try to get it
> pushed within a few hours, if there are not objections.
Sounds good to me.
> How do you feel about the other patch to rejigger the order of operations
> in CommandCounterIncrement? I think that's a bug, but it's probably
> noncritical for most people. What I'm leaning towards for that one is
> waiting till after the minor releases, then pushing it to all branches.
I've not yet have the mental cycles to look more deeply into it. I
thought your explanation why the current way is wrong made sense, but I
wanted to look a bit more into how it came to be how it is now. I agree
that pushing after the minors would make sense, it's too subtle to go
for it now.
Greetings,
Andres Freund