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 #252 and disable ignore_unknown_options only for lower version #258

Merged
merged 3 commits into from
Sep 3, 2021

Conversation

tabokie
Copy link
Member

@tabokie tabokie commented Sep 2, 2021

With #252, newer version can't be rolled back anymore. Revert it.
Also only disable ignore_unknown_options for lower version, so that newer releases of TiKV with new config items can be rolled back to this version.

This reverts commit 32b29a1.

Signed-off-by: tabokie <xy.tao@outlook.com>
Signed-off-by: tabokie <xy.tao@outlook.com>
Signed-off-by: tabokie <xy.tao@outlook.com>
@tabokie tabokie changed the title Revert #252 and strictly use ignore_unknown_options Revert #252 and disable ignore_unknown_options only for lower version Sep 2, 2021
@yiwu-arbug
Copy link
Collaborator

/run-tests

@yiwu-arbug
Copy link
Collaborator

/run-tests

@yiwu-arbug
Copy link
Collaborator

temporarily disabling mac build in CI.

Copy link
Member

@Connor1996 Connor1996 left a comment

Choose a reason for hiding this comment

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

LGTM

@tabokie tabokie merged commit 10fcdb2 into tikv:tikv-5.0 Sep 3, 2021
tabokie added a commit to tikv/rust-rocksdb that referenced this pull request Sep 3, 2021
Including following commits:
- tikv/titan#221
- tikv/titan#218
- tikv/titan#219
- tikv/rocksdb#258

Signed-off-by: tabokie <xy.tao@outlook.com>
@tabokie tabokie deleted the revert-opton branch April 20, 2022 02:48
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.

3 participants