Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

safekeeper: lift benchmarking utils into safekeeper crate #10200

Merged
merged 1 commit into from
Dec 19, 2024

Conversation

VladLazar
Copy link
Contributor

Problem

The benchmarking utilities are also useful for testing. We want to write tests in the safekeeper crate.

Summary of changes

This commit lifts the utils to the safekeeper crate. They are compiled if the benchmarking features is enabled or if in test mode.

The benchmarking utilities are also useful for testing.
We want to write tests in the safekeeper crate, so this commit
lifts the utils to the safekeeper crate. They are compiled if
the benchmarking features is enabled or if in test mode.

A small extension is also included to allow generating wal from a given
LSN. Safekeeper considers Lsn(0) invalid since it doesn't have initdb
wal.
@VladLazar VladLazar requested review from a team as code owners December 19, 2024 12:12
@VladLazar VladLazar requested review from tristan957, arssher and erikgrinaker and removed request for arssher December 19, 2024 12:12
Copy link

7095 tests run: 6796 passed, 0 failed, 299 skipped (full report)


Flaky tests (3)

Postgres 17

Postgres 16

  • test_physical_replication_config_mismatch_max_locks_per_transaction: release-arm64

Postgres 15

Code coverage* (full report)

  • functions: 31.2% (8396 of 26876 functions)
  • lines: 47.9% (66643 of 139016 lines)

* collected from Rust tests only


The comment gets automatically updated with the latest test results
8d153c5 at 2024-12-19T13:13:09.409Z :recycle:

@VladLazar VladLazar added this pull request to the merge queue Dec 19, 2024
Merged via the queue into main with commit 502d512 Dec 19, 2024
84 checks passed
@VladLazar VladLazar deleted the vlad/safekeeper-lift-bench-utils branch December 19, 2024 14:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants