| From: | Trakshan Mishra <trakshanmishra477(at)gmail(dot)com> |
|---|---|
| To: | pgsql-hackers(at)lists(dot)postgresql(dot)org |
| Cc: | tgl(at)sss(dot)pgh(dot)pa(dot)us, dean(dot)a(dot)rasheed(at)gmail(dot)com, tndrwang(at)gmail(dot)com |
| Subject: | Re: Re: merge-delete isolation test fails since 85f55534e80 |
| Date: | 2026-09-24 16:17:18 |
| Message-ID: | CACRpqq_N1rk+OQ0kw946i=uf_Lds9_aCoW0e+hjoS7+fBSjGYg@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Hi Tom,
You are right, and the fault is entirely mine. Sorry for the noise.
What was different about my installation: a stale tmp_install. The
postgres binary the isolation tests were actually executing was dated
20 September, four days old, while the expected files were being read
from a current checkout. So the server predates 85f55534e80 and does
not contain the fix, but the expected output already contained the new
WHEN NOT MATCHED BY SOURCE permutations that commit added.
What I reported as a regression is therefore the original bug that
commit fixes, not a new one. The all-NULL insert is precisely the
behaviour it corrects. I should have caught that from the symptom
alone.
A from-scratch build directory, same tree, same machine, same
configure options:
build (stale tmp_install, binary dated Sep 20) merge-delete fails
build-clean (fresh meson setup, same commit) 133/133 pass
So there is nothing wrong with the commit, and the buildfarm is right.
Apologies to Dean and Tender for the false alarm.
Regards,
Trakshan Mishra
On Thu, Sep 24, 2026 09:39 PM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> Trakshan Mishra <trakshanmishra477(at)gmail(dot)com> writes:
> > The merge-delete isolation test fails on master for me. It is not
> > intermittent -- 5 runs out of 5 -- and the failing permutations are all
> > the WHEN NOT MATCHED BY SOURCE ones.
>
> The buildfarm is happy with that, so what about your installation is
> different from everybody else?
>
> regards, tom lane
>
| From | Date | Subject | |
|---|---|---|---|
| Previous Message | Tristan Partin | 2026-09-24 16:17:07 | Re: Add counted_by attribute |