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

Revert #734 and port stats_persist_period_sec and disable_manual_compaction #735

Merged
merged 3 commits into from
Feb 2, 2023

Conversation

tabokie
Copy link
Member

@tabokie tabokie commented Feb 2, 2023

No description provided.

@tabokie tabokie requested a review from BusyJay February 2, 2023 03:40
@tabokie tabokie force-pushed the revert-periodic-work branch from 760016e to 42175e3 Compare February 2, 2023 04:44
@tabokie tabokie changed the title Revert #734 and port stats_persist_period_sec Revert #734 and port stats_persist_period_sec and disable_manual_compaction Feb 2, 2023
@BusyJay
Copy link
Member

BusyJay commented Feb 2, 2023

Should explain the motivation.

@tabokie
Copy link
Member Author

tabokie commented Feb 2, 2023

Should explain the motivation.

Read the linked tikv PR for some context. For rust-rocksdb there's no need for motivation. What's missing from the bindings should be added (eventually).

Copy link
Member

@BusyJay BusyJay left a comment

Choose a reason for hiding this comment

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

Explanation help reviewer to decide the priority of work. :)

@tabokie tabokie merged commit b2cd425 into tikv:master Feb 2, 2023
@tabokie tabokie deleted the revert-periodic-work branch February 2, 2023 07:54
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