Skip to content

[backport 3.3] wal: use existing logs sizes after restart#11307

Merged
sergepetrenko merged 1 commit intorelease/3.3from
backport/release/3.3/11098
Mar 27, 2025
Merged

[backport 3.3] wal: use existing logs sizes after restart#11307
sergepetrenko merged 1 commit intorelease/3.3from
backport/release/3.3/11098

Conversation

@TarantoolBot
Copy link
Collaborator

@TarantoolBot TarantoolBot commented Mar 27, 2025

(This PR is a backport of #11098 to release/3.3 to a future 3.3.2 release.)


During the checkpoint_wal_threshold exceedance checks tarantool used to ignore the sizes of files created prior to the server restart which resulted in cases when the server didn't create any checkpoints and generate a huge amount of logs which were replayed each server restart. Now the wal_dir is scanned for proper writer->checkpoint_wal_size calculation after the recovery process.

Fixes #9811
NO_DOC=bugfix

During the `checkpoint_wal_threshold` exceedance checks tarantool used
to ignore the sizes of files created prior to the server restart which
resulted in cases when the server didn't create any checkpoints and
generate a huge amount of logs which were replayed each server restart.
Now the `wal_dir` is scanned for proper `writer->checkpoint_wal_size`
calculation after the recovery process.

Fixes #9811
NO_DOC=bugfix

(cherry picked from commit f1ccb2f)
@TarantoolBot TarantoolBot requested a review from a team as a code owner March 27, 2025 07:34
@TarantoolBot TarantoolBot changed the title [Backport release/3.3] wal: use existing logs sizes after restart [backport 3.3] wal: use existing logs sizes after restart Mar 27, 2025
@coveralls
Copy link

Coverage Status

coverage: 87.463% (-0.005%) from 87.468%
when pulling dec7db2 on backport/release/3.3/11098
into 2a255ef
on release/3.3
.

@sergepetrenko sergepetrenko merged commit 1851df1 into release/3.3 Mar 27, 2025
25 checks passed
@sergepetrenko sergepetrenko deleted the backport/release/3.3/11098 branch March 27, 2025 10:16
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.

5 participants