| From: | Michael Paquier <michael(at)paquier(dot)xyz> |
|---|---|
| To: | Noah Misch <noah(at)leadboat(dot)com> |
| Cc: | Andres Freund <andres(at)anarazel(dot)de>, pgsql-hackers(at)lists(dot)postgresql(dot)org, Thomas Munro <thomas(dot)munro(at)gmail(dot)com>, pgbf(at)twiska(dot)com, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: XLogReadRecord() error in XlogReadTwoPhaseData() |
| Date: | 2022-01-24 02:20:21 |
| Message-ID: | Ye4M5U98cseJ6OoL@paquier.xyz |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Sun, Jan 23, 2022 at 06:10:07PM -0800, Noah Misch wrote:
> Could do that. Every run that doesn't get the flaky failure will print a
> message like "TODO passed: 3-5", though the test file could mitigate that by
> declaring the TODO only on configurations where we expect a failure. The
> 027_stream_regress.pl trouble involves reaching a die(), not failing a test in
> this sense, so that one would take more work.
Using a TODO has the advantage to allow the tests to run on a periodic
basis, even if they could fail in this unexpected way. We've had our
load of issues in the past proper to specific architectures, so this
could help with the 2PC code paths and SPARC. Not to mention that we
could just let the code be as-is, and it would fix itself once the
kernel is updated. So that sounds much better to me than a skip
phase.
--
Michael
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Andres Freund | 2022-01-24 02:24:28 | Re: Replace uses of deprecated Python module distutils.sysconfig |
| Previous Message | Noah Misch | 2022-01-24 02:10:07 | Re: XLogReadRecord() error in XlogReadTwoPhaseData() |