On Wed, Mar 18, 2026 at 4:44 PM Tom Lane <[email protected]> wrote: > Don't know if you noticed yet, but avocet has shown [1] that one > pg_plan_advice test case is unstable under debug_discard_caches = 1:
I had not. Thanks for the pointer. > It looks like the appearance of "Supplied Plan Advice:" depends > on whether the prepared query's plan got regenerated or not. > I'm not sure if this represents a bug (ie undesirable behavior) or > it's just that the test is being insufficiently careful about > being reproducible. Well, that's embarrassing: it's a copy-and-paste error. The test prepares and executes pt1, then prepares and executes pt2, then prepares pt3 and executes pt1, then prepares pt4 and execute p2. pt3 and pt4 are never used for anything. Also there's a related typo in a comment. See attached. -- Robert Haas EDB: http://www.enterprisedb.com
v1-0001-pg_plan_advice-Fix-multiple-copy-and-paste-errors.nocfbot
Description: Binary data
