Skip to content

Fix VDisk compaction bug (merge from main #16151) - #16194

Merged
alexvru merged 1 commit into
ydb-platform:stable-25-1-1from
alexvru:merge/25-1-1/16151
Mar 25, 2025
Merged

alexvru merged 1 commit into
ydb-platform:stable-25-1-1from
alexvru:merge/25-1-1/16151

Conversation

@alexvru

@alexvru alexvru commented Mar 25, 2025

Copy link
Copy Markdown
Contributor

Changelog entry

Fix VDisk compaction bug

Changelog category

  • Bugfix

Description for reviewers

This fixes VDisk compaction bug when multiple L0 tables are generated during re-compaction of existing L0.

#16123

@alexvru
alexvru requested a review from a team as a code owner March 25, 2025 08:54
@github-actions

github-actions Bot commented Mar 25, 2025 •

Copy link
Copy Markdown
Contributor

⚪ 2025-03-25 08:56:51 UTC Pre-commit check linux-x86_64-relwithdebinfo for fb9e43a has started.
⚪ 2025-03-25 08:57:07 UTC Artifacts will be uploaded here
⚪ 2025-03-25 09:00:06 UTC ya make is running...
🟡 2025-03-25 10:01:21 UTC Some tests failed, follow the links below. Going to retry failed tests...

Details

Test history | Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
19431 18078 0 4 1216 133

⚪ 2025-03-25 10:03:12 UTC ya make is running... (failed tests rerun, try 2)
🟢 2025-03-25 10:30:52 UTC Tests successful.

Test history | Ya make output | Test bloat | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
207 (only retried tests) 75 0 0 1 131

🟢 2025-03-25 10:31:01 UTC Build successful.
🟢 2025-03-25 10:31:26 UTC ydbd size 2.1 GiB changed* by -2.6 KiB, which is <= 0 Bytes vs stable-25-1-1: OK

ydbd size dash stable-25-1-1: 0115065 merge: fb9e43a diff diff %
ydbd size 2 242 407 112 Bytes 2 242 404 424 Bytes -2.6 KiB -0.000%
ydbd stripped size 474 916 952 Bytes 474 916 056 Bytes -896 Bytes -0.000%

*please be aware that the difference is based on comparing your commit and the last completed build from the post-commit, check comparation

@github-actions

github-actions Bot commented Mar 25, 2025 •

Copy link
Copy Markdown
Contributor

⚪ 2025-03-25 08:58:51 UTC Pre-commit check linux-x86_64-release-asan for fb9e43a has started.
⚪ 2025-03-25 08:59:05 UTC Artifacts will be uploaded here
⚪ 2025-03-25 09:01:54 UTC ya make is running...
🟡 2025-03-25 10:13:06 UTC Some tests failed, follow the links below. This fail is not in blocking policy yet Going to retry failed tests...

Details

Test history | Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
12099 12032 0 20 11 36

⚪ 2025-03-25 10:14:11 UTC ya make is running... (failed tests rerun, try 2)
🟡 2025-03-25 10:34:41 UTC Some tests failed, follow the links below. This fail is not in blocking policy yet Going to retry failed tests...

Details

Test history | Ya make output | Test bloat | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
101 (only retried tests) 65 0 1 0 35

⚪ 2025-03-25 10:34:49 UTC ya make is running... (failed tests rerun, try 3)
🟢 2025-03-25 10:54:59 UTC Tests successful.

Test history | Ya make output | Test bloat | Test bloat | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
65 (only retried tests) 33 0 0 1 31

🟢 2025-03-25 10:55:06 UTC Build successful.
🟢 2025-03-25 10:55:33 UTC ydbd size 3.6 GiB changed* by -5.6 KiB, which is <= 0 Bytes vs stable-25-1-1: OK

ydbd size dash stable-25-1-1: 0115065 merge: fb9e43a diff diff %
ydbd size 3 900 702 336 Bytes 3 900 696 576 Bytes -5.6 KiB -0.000%
ydbd stripped size 1 365 201 200 Bytes 1 365 200 560 Bytes -640 Bytes -0.000%

*please be aware that the difference is based on comparing your commit and the last completed build from the post-commit, check comparation

@alexvru
alexvru enabled auto-merge (squash) March 25, 2025 09:00
@alexvru
alexvru merged commit 017624b into ydb-platform:stable-25-1-1 Mar 25, 2025
@alexvru
alexvru deleted the merge/25-1-1/16151 branch March 25, 2025 16:34
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