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

Fixup: raft-kv-metastore example, Arc<Store>::purge_logs_upto issues wrong debug message #914

Closed
wants to merge 1 commit into from

Conversation

pfwang80s
Copy link
Contributor

@pfwang80s pfwang80s commented Oct 24, 2023

purge_logs_upto is purging (-oo, log_id], not [log_id, +oo).


This change is Reviewable

@drmingdrmer
Copy link
Member

Thank you!

The commit message should starts with Fixup: or Chore: , or the commit message linter does not pass.

@pfwang80s pfwang80s changed the title fix a typo that debug! issues wrong message Fixup: fix a typo that debug! issues wrong message Oct 25, 2023
@drmingdrmer
Copy link
Member

Thank you!

The commit message should starts with Fixup: or Chore: , or the commit message linter does not pass.

The title of the commit message is what truly matters, not the title of the pull request.

@pfwang80s pfwang80s changed the title Fixup: fix a typo that debug! issues wrong message Fixup: raft-kv-metastore example, Arc<Store>::purge_logs_upto issues wrong debug message Oct 26, 2023
@drmingdrmer
Copy link
Member

What I mean is update the commit message to something like this.
Do not worry, I'll manually merge this commit.
image

@drmingdrmer
Copy link
Member

Manually merged in main branch:) 0f39431

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