On Fri, Feb 21, 2025 at 3:36 PM Amit Langote <amitlangot...@gmail.com> wrote: > On Fri, Feb 21, 2025 at 3:04 PM Tom Lane <t...@sss.pgh.pa.us> wrote: > > > > Amit Langote <amitlangot...@gmail.com> writes: > > > I pushed the final piece yesterday. > > > > trilobite reports that this fails under -DCLOBBER_CACHE_ALWAYS: > > > > https://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=trilobite&dt=2025-02-20%2019%3A37%3A12 > > Looking, thanks for the heads up.
The short of it is that the cached-plan-inval test in the delay_execution suite can never be made to work under CLOBBER_CACHE_ALWAYS. The test assumes that locks on partitions for a reused generic plan are not taken until InitPlan(). However, under CLOBBER_CACHE_ALWAYS, generic plans are never reused, so the test's assumption never holds. I see two possible ways to address this: 1. Find a way to disable the cached-plan-inval test in CLOBBER_CACHE_ALWAYS builds. However, I haven't found any other test that does this. 2. Remove the test altogether, though that might be too drastic. Thoughts? -- Thanks, Amit Langote
