-
Notifications
You must be signed in to change notification settings - Fork 312
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
server: support app env rocksdb.checkpoint.reserve_min_count and rock… #252
Conversation
…sdb.checkpoint.reserve_time_seconds
} | ||
} | ||
|
||
if (count != _checkpoint_reserve_min_count) { |
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.
怎么都没有使用从envs
中读到的值?
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.
count就是啊
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.
看花眼了。。
不过count和time,如果其中一个解析成功,另一个失败(这时实际是in_config中的值,或者一个负值),但同时都不等于当前值,是不是都要去设置?
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.
有道理,还是同时成功才能生效吧
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.
嗯,这个我之前也没考虑到。。。
…sdb.checkpoint.reserve_time_seconds (#252)
…sdb.checkpoint.reserve_time_seconds (apache#252) Former-commit-id: 82d9b1664f3ff1c1fe47b68f2bc7459f9ee17495 [formerly f8d8060] Former-commit-id: 7ac4a9bff3276cf4240fe97155a030fefa928236
…sdb.checkpoint.reserve_time_seconds
for the reason, please refer to Rocksdb-Checkpoint管理