Skip to content

Commit

Permalink
Do not perform prefetch for temp relations (#10146)
Browse files Browse the repository at this point in the history
## Problem

See https://neondb.slack.com/archives/C04DGM6SMTM/p1734002916827019

With recent prefetch fixes for pg17 and `effective_io_concurrency=100` 
pg_regress test stats.sql is failed when set temp_buffers to 100.
Stream API will try to lock all this 100 buffers for prefetch.

## Summary of changes

Disable such behaviour for temp relations.
Postgres PR: neondatabase/postgres#548

Co-authored-by: Konstantin Knizhnik <[email protected]>
  • Loading branch information
knizhnik and Konstantin Knizhnik authored Dec 16, 2024
1 parent f3ecd5d commit 117c1b5
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion vendor/postgres-v17
2 changes: 1 addition & 1 deletion vendor/revisions.json
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
{
"v17": [
"17.2",
"010c0ea2eb06afe76485a33c43954cbcf3d99f86"
"65c4e46baf56ec05412c7dd63d62faff0b33dcfb"
],
"v16": [
"16.6",
Expand Down

1 comment on commit 117c1b5

@github-actions
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

7084 tests run: 6786 passed, 1 failed, 297 skipped (full report)


Failures on Postgres 17

# Run all failed tests locally:
scripts/pytest -vv -n $(nproc) -k "test_timeline_archival_chaos[release-pg17]"
Flaky tests (5)

Postgres 17

Postgres 16

  • test_pgdata_import_smoke[None-1024-RelBlockSize.MULTIPLE_RELATION_SEGMENTS]: release-arm64

Test coverage report is not available

The comment gets automatically updated with the latest test results
117c1b5 at 2024-12-16T06:57:04.987Z :recycle:

Please sign in to comment.