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

monitor: expose rocksdb statistics to pegasus #212

Merged
merged 12 commits into from
Nov 13, 2018

Conversation

acelyc111
Copy link
Member

Copy link
Contributor

@qinzuoyan qinzuoyan left a comment

Choose a reason for hiding this comment

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

另外更新rdsn到最新

src/server/pegasus_server_impl.cpp Outdated Show resolved Hide resolved
src/shell/commands.h Outdated Show resolved Hide resolved
<< std::setw(w) << std::right << "abnormal" << std::setw(w) << std::right << "file_mb"
<< std::setw(w) << std::right << "file_num";
<< std::setw(w) << std::right << "abnormal" << std::setw(w) << std::right
<< "file_size(MB)" << std::setw(w) << std::right << "file_num" << std::setw(w)
Copy link
Contributor

Choose a reason for hiding this comment

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

file_size(MB) -> file_mb
mem_usage(MB) -> memtbl_mb

Copy link
Member Author

Choose a reason for hiding this comment

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

mem_usage(MB) -> memtbl_mb改成rdb_mem_mb吧,他除了包含memtable,还包含index_and_filter_blocks占用的内存

src/shell/commands.h Show resolved Hide resolved
src/server/pegasus_server_impl.cpp Show resolved Hide resolved
src/server/pegasus_server_impl.cpp Show resolved Hide resolved
src/server/pegasus_server_impl.cpp Outdated Show resolved Hide resolved
src/shell/commands.h Outdated Show resolved Hide resolved
src/server/pegasus_server_impl.cpp Outdated Show resolved Hide resolved
qinzuoyan
qinzuoyan previously approved these changes Nov 13, 2018
_updating_rocksdb_sstsize_timer_task = nullptr;
if (_update_replica_rdb_stat != nullptr) {
_update_replica_rdb_stat->cancel(true);
_update_replica_rdb_stat = nullptr;
}
Copy link
Contributor

Choose a reason for hiding this comment

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

新加的 _update_server_rdb_stat 是不是也要 cancel ?

Copy link
Member Author

Choose a reason for hiding this comment

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

_update_server_rdb_stat是全局的,不cancel,他的生命周期跟进程对应,而_update_replica_rdb_stat生命周期跟replica对应

@qinzuoyan
Copy link
Contributor

对了,忘记说了:更新下update screen脚本,把新counter加入到screen里面?

@qinzuoyan qinzuoyan merged commit 9e58867 into apache:master Nov 13, 2018
neverchanje pushed a commit to neverchanje/pegasus that referenced this pull request Jul 13, 2019
Former-commit-id: 33104d37ded07ba4a8827e70a3624a61822599fb [formerly 9e58867]
Former-commit-id: 24398d7ac3a9944138b3aea065150353b55aadf2
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