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

scripts: improve pegasus_rolling_update.sh; fix shell #167

Merged
merged 3 commits into from
Aug 20, 2018
Merged

Conversation

qinzuoyan
Copy link
Contributor

No description provided.

@qinzuoyan qinzuoyan added the type/enhancement Indicates new feature requests label Aug 20, 2018
@@ -3654,10 +3654,9 @@ inline bool app_stat(command_executor *e, shell_context *sc, arguments args)
<< std::setw(w) << std::right << "INCR" << std::setw(w) << std::right << "CAS"
<< std::setw(w) << std::right << "CAM" << std::setw(w) << std::right << "SCAN";
if (!only_qps) {
out << std::setw(w) << std::right << "storage_mb" << std::setw(w) << std::right
Copy link
Contributor

Choose a reason for hiding this comment

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

为啥不打印 storage_mb ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

因为storage_mb改成了file_mb,已经打印了

@qinzuoyan qinzuoyan merged commit b3a941c into master Aug 20, 2018
@qinzuoyan qinzuoyan deleted the qinzuoyan branch August 20, 2018 07:11
@neverchanje neverchanje added component/scripts component/shell pegasus shell and removed type/enhancement Indicates new feature requests labels Sep 13, 2018
neverchanje pushed a commit to neverchanje/pegasus that referenced this pull request Jul 13, 2019
Former-commit-id: 3b411ffba9144efb362d118cd8722482d86179cd [formerly b3a941c]
Former-commit-id: e151dbf18ef572b28546e48ce9b628cf137d5a04
cauchy1988 pushed a commit to cauchy1988/incubator-pegasus that referenced this pull request May 7, 2022
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
component/shell pegasus shell
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants