test(bottom-most-compaction): wal apply order - Githubissues

neondatabase / neon

Neon: Serverless Postgres. We separated storage and compute to offer autoscaling, code-like database branching, and scale to zero.

https://neon.tech

Apache License 2.0

13.18k stars 367 forks source link

test(bottom-most-compaction): wal apply order #8163

Closed skyzh closed 2 days ago

skyzh commented 3 days ago

Problem

A follow-up on https://github.com/neondatabase/neon/pull/8103/. Current main branch fails with:

assertion `left == right` failed
  left: b"value 3@0x10@0x30@0x28@0x40"
 right: b"value 3@0x10@0x28@0x30@0x40"

will be fixed after #8103 gets merged.

Summary of changes

Add multiple WALs below GC horizon above base image for key 3

Checklist before requesting a review

[x] I have performed a self-review of my code.
[x] If it is a core feature, I have added thorough tests.
[ ] Do we need to implement analytics? if so did you add the relevant metrics to the dashboard?
[ ] If this PR requires public announcement, mark it with /release-notes label and add several sentences in this section.

Checklist before merging

[ ] Do not forget to reformat commit message to not include the above checklist

github-actions[bot] commented 3 days ago

2910 tests run: 2794 passed, 0 failed, 116 skipped (full report)

Code coverage* (full report)

functions: 32.8% (6891 of 21016 functions)
lines: 50.3% (53895 of 107181 lines)

* collected from Rust tests only

_{The comment gets automatically updated with the latest test results
792183ae0ef4f1f8b22e9ac7e8748740ab73f873 at 2024-06-26T01:07:22.145Z :recycle:}