Skip to content

Conversation

@osalyk
Copy link
Contributor

@osalyk osalyk commented Jan 14, 2026

This change is Reviewable

Copy link
Contributor

@janekmi janekmi left a comment

Choose a reason for hiding this comment

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

@janekmi reviewed all commit messages and made 1 comment.
Reviewable status: 0 of 11 files reviewed, 1 unresolved discussion (waiting on @grom72 and @osalyk).


src/test/pmempool_sync/TEST43 line 59 at r1 (raw file):

create_recovery_file $DIR/testset1_r1_p2_badblocks.txt

turn_on_checking_bad_blocks $POOLSET

Can we remove all of these in one go?

Copy link
Contributor Author

@osalyk osalyk left a comment

Choose a reason for hiding this comment

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

@osalyk made 1 comment.
Reviewable status: 0 of 16 files reviewed, 1 unresolved discussion (waiting on @grom72 and @janekmi).


src/test/pmempool_sync/TEST43 line 59 at r1 (raw file):

Previously, janekmi (Jan Michalski) wrote…

Can we remove all of these in one go?

Done.

Copy link
Contributor

@janekmi janekmi left a comment

Choose a reason for hiding this comment

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

@janekmi reviewed 16 files and all commit messages, made 1 comment, and resolved 1 discussion.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @grom72).


a discussion (no related file):
Whoever will merge this it would be good to mention #28 in the commit message.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants