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.

Sign up for GitHub

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

Jump to bottom

[docdb] Rocksdb num SST files info missing from TS /tablets view #7647

Closed
bmatican opened this issue Mar 15, 2021 · 2 comments
Closed

[docdb] Rocksdb num SST files info missing from TS /tablets view #7647

bmatican opened this issue Mar 15, 2021 · 2 comments
Assignees
@jmeehan16
Labels
area/docdb YugabyteDB core features
Projects
YBase features

Comments

@bmatican
Copy link
Contributor

bmatican commented Mar 15, 2021 •
edited

This is the header and a sample first row from a TS_IP:9000/tablets

Screen Shot 2021-03-15 at 11 48 28 AM

Note how the number of SST column is empty for some reason.

Funny enough, in the /tables view, where we aggregate across all relevant tablets, that is present, so the data must be easily accessible!

cc @robertsami @ttyusupov
@bmatican bmatican added the area/docdb YugabyteDB core features label Mar 15, 2021
@jmeehan16
Copy link
Contributor

Fix incoming, pending review. Below is a screen capture post-fix:
tserver-tablets-7647-fix

jmeehan16 added a commit that referenced this issue Mar 19, 2021
@jmeehan16
[#7647] docdb: Adds Num SST Files to TS tablets view
7413808 
Summary: The order of printed outputs for each entry in the TServer /tablets page was mixed up, resulting in an incorrect result for the State and Num SST Files columns. This rectifies that and makes the function a little easier to read.

Test Plan: Screenshot of the post-fix /tablets view is attached [[ #7647 (comment) | screenshot ]]

Reviewers: bogdan

Reviewed By: bogdan

Subscribers: ybase, timur, rsami

Differential Revision: https://phabricator.dev.yugabyte.com/D10970
@jmeehan16
Copy link
Contributor

Commit 7413808 resolves this issue.

YintongMa pushed a commit to YintongMa/yugabyte-db that referenced this issue May 26, 2021
@jmeehan16 @YintongMa
[yugabyte#7647] docdb: Adds Num SST Files to TS tablets view
cc3c047 
Summary: The order of printed outputs for each entry in the TServer /tablets page was mixed up, resulting in an incorrect result for the State and Num SST Files columns. This rectifies that and makes the function a little easier to read.

Test Plan: Screenshot of the post-fix /tablets view is attached [[ yugabyte#7647 (comment) | screenshot ]]

Reviewers: bogdan

Reviewed By: bogdan

Subscribers: ybase, timur, rsami

Differential Revision: https://phabricator.dev.yugabyte.com/D10970
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jmeehan16 jmeehan16

Labels
area/docdb YugabyteDB core features
Projects
YBase features
  
Awaiting triage
Milestone
No milestone
Development

No branches or pull requests
2 participants
@bmatican @jmeehan16