Skip to content

Commit

Permalink
Fix flaky append regression test take 2
Browse files Browse the repository at this point in the history
In timescale#7512 we added manual `VACUUM` in all involved relations to avoid
flaky output tests.

But looks like it is not working very well specially on Windows builds,
so now we're replacing the output `Heap Fetches: [0-9]` for `Heap
Fetches: 0` to make have a more predictable output test.
  • Loading branch information
fabriziomello committed Dec 5, 2024
1 parent 396caf6 commit adb3288
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion test/runner_cleanup_output.sh
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,8 @@ RUNNER=${1:-""}
sed -e '/<exclude_from_test>/,/<\/exclude_from_test>/d' \
-e 's! Memory: [0-9]\{1,\}kB!!' \
-e 's! Memory Usage: [0-9]\{1,\}kB!!' \
-e 's! Average Peak Memory: [0-9]\{1,\}kB!!' | \
-e 's! Average Peak Memory: [0-9]\{1,\}kB!!' \
-e 's! Heap Fetches: [0-9]\{1,\}! Heap Fetches: 0!' | \
grep -v 'DEBUG: rehashing catalog cache id' | \
grep -v 'DEBUG: compacted fsync request queue from' | \
grep -v 'DEBUG: creating and filling new WAL file' | \
Expand Down

0 comments on commit adb3288

Please sign in to comment.