Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Corrected an issue on the "NuoDB Ops System Overview" databoard that resulted in the incorrect display of TE counts and aggregate TPS values in the Aggregate TPS panel. #29

Merged
merged 2 commits into from
Apr 30, 2021

Conversation

butson
Copy link
Contributor

@butson butson commented Apr 30, 2021

This is a fix to the aggregate panel. Please test. My limited testing is okay. I did not review / change any other panel

@mkysel mkysel requested review from bkelly-ndb and jleslie85 and removed request for acabrele and sivanov-nuodb April 30, 2021 15:10
@mkysel mkysel added the bug Something isn't working label Apr 30, 2021
…variable pid should be removed and just use startid. This is the only dashboard that does this, I think. But, for now left it.
@jleslie85 jleslie85 changed the title fixed aggregated panel Corrected an issue on the "NuoDB Ops System Overview" databoard that resulted in incorrect display of TE counts and aggregate TPS values in the Aggregate TPS panel. Apr 30, 2021
@jleslie85 jleslie85 changed the title Corrected an issue on the "NuoDB Ops System Overview" databoard that resulted in incorrect display of TE counts and aggregate TPS values in the Aggregate TPS panel. Corrected an issue on the "NuoDB Ops System Overview" databoard that resulted in the incorrect display of TE counts and aggregate TPS values in the Aggregate TPS panel. Apr 30, 2021
@mkysel mkysel merged commit 157add0 into master Apr 30, 2021
mkysel pushed a commit that referenced this pull request Apr 30, 2021
…resulted in the incorrect display of TE counts and aggregate TPS values in the Aggregate TPS panel. (#29)
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants