forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commitd6c0db1
committed
pgstat: Hide instability in stats.spec with -DCATCACHE_FORCE_RELEASE.
With -DCATCACHE_FORCE_RELEASE a few tests failed. Those were trying to testbehavior in the absence of invalidation processing and-DCATCACHE_FORCE_RELEASE obviously adds a lot of invalidation processing. Thetest already tried to handle debug_discard_caches > 0, by disabling it forindividual tests.Instead hide potentially problematic function calls in a wrapper function thatcatches the does-not-exist error. The error isn't the actually interestingbit, it's whether the stats entry still exist afterwards.I confirmed that the tests still catches leaked function stats if I nuke theprotections against that in pgstat_function.c.Per buildfarm animal prion.Discussion:https://postgr.es/m/20220407165709.jgdkrzqlkcwue6ko@alap3.anarazel.de1 parent5264add commitd6c0db1
File tree
3 files changed
+172
-170
lines changed- src/test/isolation
- expected
- specs
3 files changed
+172
-170
lines changed0 commit comments
Comments
(0)