-
Notifications
You must be signed in to change notification settings - Fork 92
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
disable ignore_unknown_options only for lower version #256
Conversation
Signed-off-by: tabokie <xy.tao@outlook.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Though we still can't downgrade to lower version for previous version
It seem unsolvable for older releases, we can only make sure 5.2 can rollback to upcoming minor versions of 5.0 and 5.1, and then rollback to older releases because Options file will be updated without unrecognized items. I'll also revert the disable-write-stall config for 5.0 and 5.1 later. |
Shall we ignore unknown options only when current version is no less than the previous version, same as what #258 did? |
Signed-off-by: tabokie <xy.tao@outlook.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Only enable force option check when reading options file from lower version. Signed-off-by: tabokie <xy.tao@outlook.com>
Only enable force option check when reading options file from lower version. Signed-off-by: tabokie <xy.tao@outlook.com>
Only enable force option check when reading options file from lower version. Signed-off-by: tabokie <xy.tao@outlook.com>
Only enable force option check when reading options file from lower version. Signed-off-by: tabokie <xy.tao@outlook.com>
Only enable force option check when reading options file from lower version. Signed-off-by: tabokie <xy.tao@outlook.com>
Only enable force option check when reading options file from lower version. Signed-off-by: tabokie <xy.tao@outlook.com>
…)" This reverts commit a06cd61. Signed-off-by: tabokie <xy.tao@outlook.com>
…)" This reverts commit c92ff87. Signed-off-by: tabokie <xy.tao@outlook.com>
* disable ignore_unknown_options only for lower version (#256) Only enable force option check when reading options file from lower version. Signed-off-by: tabokie <xy.tao@outlook.com> * fix CreateDBStatistics visibility issue Signed-off-by: tabokie <xy.tao@outlook.com> * remove blob index access during compaction Signed-off-by: tabokie <xy.tao@outlook.com> * fix build issues Signed-off-by: tabokie <xy.tao@outlook.com> * restore check but tolerate failure Signed-off-by: tabokie <xy.tao@outlook.com> * panic when calling BlobIndex::DecodeFrom Signed-off-by: tabokie <xy.tao@outlook.com> * Revert "disable ignore_unknown_options only for lower version (#256)" This reverts commit a06cd61. Signed-off-by: tabokie <xy.tao@outlook.com> * Revert "panic when calling BlobIndex::DecodeFrom" This reverts commit 53f6fb0. Signed-off-by: tabokie <xy.tao@outlook.com>
Signed-off-by: tabokie xy.tao@outlook.com
Only enable force option check when reading options file from lower version.