Refactor the TAP test of auto_explain
authorMichael Paquier <[email protected]>
Fri, 1 Jul 2022 00:13:57 +0000 (09:13 +0900)
committerMichael Paquier <[email protected]>
Fri, 1 Jul 2022 00:13:57 +0000 (09:13 +0900)
commit550bc0a6c0c6af20d34c16902f0ed977550396eb
tree3bda57371ba58c33da8ddf2fa4d437b8ad4cfebf
parent258f48f858b0bcc4c3fac3fa01f79ee61ad0cd1e
Refactor the TAP test of auto_explain

Previously, the tests were structured so as all the queries whose plans
are checked run first, followed by pattern checks using the full set of
server logs.  This can be problematic when extending the tests, as this
increases query plan overlaps, where two tests finish by having similar
plan outputs potentially invalidating the tests wanted.

The tests are refactored so as log content matches are checked in
isolation of each query run, by grabbing the position of the server logs
before running each query whose plan is generated in the logs.  This
avoids issues when extending the tests, something that would become a
larger problem with a follow-up patch that adds a new GUC in
auto_explain to control the size of the each parameter logged.

Author: Dagfinn Ilmari Mannsåker
Discussion: https://postgr.es/m/[email protected]
contrib/auto_explain/t/001_auto_explain.pl