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

server: support app env rocksdb.checkpoint.reserve_min_count and rock… #252

Merged
merged 3 commits into from
Jan 10, 2019

Conversation

qinzuoyan
Copy link
Contributor

…sdb.checkpoint.reserve_time_seconds

for the reason, please refer to Rocksdb-Checkpoint管理

@qinzuoyan qinzuoyan added the type/enhancement Indicates new feature requests label Jan 10, 2019
@qinzuoyan qinzuoyan self-assigned this Jan 10, 2019
hycdong
hycdong previously approved these changes Jan 10, 2019
}
}

if (count != _checkpoint_reserve_min_count) {
Copy link
Member

Choose a reason for hiding this comment

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

怎么都没有使用从envs中读到的值?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

count就是啊

Copy link
Member

Choose a reason for hiding this comment

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

看花眼了。。
不过count和time,如果其中一个解析成功,另一个失败(这时实际是in_config中的值,或者一个负值),但同时都不等于当前值,是不是都要去设置?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

有道理,还是同时成功才能生效吧

Copy link
Contributor

Choose a reason for hiding this comment

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

嗯,这个我之前也没考虑到。。。

hycdong
hycdong previously approved these changes Jan 10, 2019
@qinzuoyan qinzuoyan merged commit f8d8060 into master Jan 10, 2019
@qinzuoyan qinzuoyan deleted the qinzuoyan branch January 10, 2019 11:35
qinzuoyan added a commit that referenced this pull request Feb 27, 2019
neverchanje pushed a commit to neverchanje/pegasus that referenced this pull request Jul 13, 2019
…sdb.checkpoint.reserve_time_seconds (apache#252)

Former-commit-id: 82d9b1664f3ff1c1fe47b68f2bc7459f9ee17495 [formerly f8d8060]
Former-commit-id: 7ac4a9bff3276cf4240fe97155a030fefa928236
acelyc111 pushed a commit that referenced this pull request Jun 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/enhancement Indicates new feature requests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants