Skip to content

Fix flaky stats check in vacuum_progress_row/column tests - #2034

Open
Alena0704 wants to merge 1 commit into
apache:REL_2_STABLEfrom
Alena0704:fix-vacuum-progress-stats-wait-rel2
Open

Alena0704 wants to merge 1 commit into
apache:REL_2_STABLEfrom
Alena0704:fix-vacuum-progress-stats-wait-rel2

Conversation

@Alena0704

@Alena0704 Alena0704 commented Sep 18, 2026

Copy link
Copy Markdown
Collaborator

The tests read pg_stat_all_tables right after the DELETE, while the stats of the aborted inserts may not have arrived yet and those of the DELETE may already have. Wait for the stats of the aborts and read the view before the DELETE instead.

Type of Change

  • Bug fix (non-breaking change)
  • New feature (non-breaking change)
  • Breaking change (fix or feature with breaking changes)
  • Documentation update

Breaking Changes

Test Plan

  • Unit tests added/updated
  • Integration tests added/updated
  • Passed make installcheck
  • Passed make -C src/test installcheck-cbdb-parallel

Impact

Performance:

User-facing changes:

Dependencies:

Checklist

Additional Context

CI Skip Instructions


The tests read pg_stat_all_tables right after the DELETE, while the
stats of the aborted inserts may not have arrived yet and those of the
DELETE may already have.  Wait for the stats of the aborts and read the
view before the DELETE instead.
@Alena0704
Alena0704 marked this pull request as ready for review September 18, 2026 20:02
@Alena0704 Alena0704 added type: Testing Adding missing tests or correcting existing tests CI labels Sep 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI type: Testing Adding missing tests or correcting existing tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant