Skip to content
This repository has been archived by the owner on Jun 23, 2022. It is now read-only.

feat(online_migration):part1 - add new app_envs to control rocksdb allow_ingest_behind #991

Merged
merged 7 commits into from
Dec 29, 2021

Conversation

hycdong
Copy link
Contributor

@hycdong hycdong commented Dec 17, 2021

As apache/incubator-pegasus#851 show, we decide to support online migration.
This pull request adds a new app_envs to control rocksdb allow_ingest_behind option, inluding:

  1. Add a new app_envs rocksdb.allow_ingest_behind
  2. This app_envs will also parse to replica server through on_config_sync, and update in update_allow_ingest_behind
  3. The value of app_envs rocksdb.allow_ingest_behind will be store in .app_info, see in function replica_app_info::store

@hycdong hycdong marked this pull request as draft December 17, 2021 09:15
@hycdong hycdong marked this pull request as ready for review December 17, 2021 09:50
Smityz
Smityz previously approved these changes Dec 20, 2021
Smityz
Smityz previously approved these changes Dec 23, 2021
src/replica/replica.h Outdated Show resolved Hide resolved
@hycdong hycdong merged commit 08af400 into XiaoMi:master Dec 29, 2021
@hycdong hycdong deleted the add_app_envs_ingest_behind branch December 29, 2021 10:55
hycdong added a commit to hycdong/rdsn that referenced this pull request Jan 29, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants