Skip to content

Commit

Permalink
Do not assign max(lsn) to maxLastWrittenLsn in SetLastWrittenLSNForbl…
Browse files Browse the repository at this point in the history
…okv (#10474)

## Problem

See #10281

`SetLastWrittenLSNForBlockv` is assigning max(lsn) to
`maxLastWrittenLsn` while its should contain only max LSN not present in
LwLSN cache. It case unnecessary waits in PS.

## Summary of changes

Restore status-quo for pg17.

Related Postgres PR: neondatabase/postgres#563

---------

Co-authored-by: Konstantin Knizhnik <knizhnik@neon.tech>
  • Loading branch information
knizhnik and Konstantin Knizhnik authored Jan 24, 2025
1 parent 7000aaa commit 9f1408f
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",
"4276717f6e91023e504de355f4f21d4824074de8"
"46f9b96555e084c35dd975da9485996db9e86181"
],
"v16": [
"16.6",
Expand Down

1 comment on commit 9f1408f

@github-actions
Copy link

@github-actions github-actions bot commented on 9f1408f Jan 24, 2025

Choose a reason for hiding this comment

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

7559 tests run: 7161 passed, 1 failed, 397 skipped (full report)


Failures on Postgres 16

# Run all failed tests locally:
scripts/pytest -vv -n $(nproc) -k "test_layer_map[release-pg16-github-actions-selfhosted]"
Flaky tests (7)

Postgres 17

Postgres 16

Postgres 14

Code coverage* (full report)

  • functions: 33.5% (8496 of 25347 functions)
  • lines: 49.3% (71463 of 144982 lines)

* collected from Rust tests only


The comment gets automatically updated with the latest test results
9f1408f at 2025-01-24T18:26:59.272Z :recycle:

Please sign in to comment.