-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
support expression in update configs #1273
Conversation
Unit testing failed. |
Unit testing passed. |
Unit testing passed. |
return Status::Error("Invalid string config value"); | ||
} | ||
|
||
buf += Expression::toString(ret.value()); |
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.
Like this !
Unit testing passed. |
1 similar comment
Unit testing passed. |
Unit testing passed. |
Unit testing passed. |
Changing the log level is the most frequently used command. Definitely, we should provide a document about this operation. could you give an example doc. So I can abandon #1235 |
Maybe we can still keep #1235, because if user have run meta before, it will register configs of Anyway, I'll give a brief sample of update |
55e1104
to
e4cb3aa
Compare
Unit testing passed. |
Unit testing passed. |
Unit testing failed. |
Jenkins go |
Unit testing passed. |
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.
Nice ~
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, good job!
Unit testing passed. |
Unit testing passed. |
* support expression in update configs * fix ut * Return error of string option * remove some rocksdb configs, support change more configs in console * remove change in default template * fix ut * update gflags.json
* support expression in update configs * fix ut * Return error of string option * remove some rocksdb configs, support change more configs in console * remove change in default template * fix ut * update gflags.json
* Add meta offline detect * fix conflict Co-authored-by: Alex Xing <90179377+SuperYoko@users.noreply.github.com>
update configs storage:rocksdb_column_family_options={ target_file_size_base=100*1024*1024 }
write_buffer_size
inupdate configs
, it would cause rocksdb crash. Remove some configs which can't be set dynamically.minlevel
,v
, etc.