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

"transactions" table should be marked as system table #874

Closed
ndeodhar opened this issue Feb 15, 2019 · 1 comment
Closed

"transactions" table should be marked as system table #874

ndeodhar opened this issue Feb 15, 2019 · 1 comment
Labels
kind/enhancement This is an enhancement of an existing feature

Comments

@ndeodhar
Copy link
Contributor

ndeodhar commented Feb 15, 2019 •
edited

Currently, transactions table shows up as a "user table" in UI, which is incorrect. This should be marked as a system table instead.
@ndeodhar ndeodhar changed the title "transactions table should be marked as system table "transactions" table should be marked as system table Feb 15, 2019
@kmuthukk kmuthukk added the kind/enhancement This is an enhancement of an existing feature label Feb 17, 2019
yugabyte-ci pushed a commit that referenced this issue Mar 13, 2019
@ndeodhar
ENG-4683 (#874): Fix user table UI to exclude transactions and sequen…
9093416 
…ces tables

Summary:
Currently, transactions table shows up as a "user table" in UI, which is incorrect. This should be marked as a system table instead.

Changes include:
1. Remove transactions table from the User Table section
2. Change the column names in the TServer list
{{Load (Num Tablets) --> User Tablets (Total)
Leader Count (Num Tablets) --> User Tablets (Leaders)}}
3. Add 2 new columns to the end of the TServer list
{{System Tablets (Total)
System Tablets (Leaders)}}
4. Move UUID in the Tserver list to right below the name of the TServer in the Server column.
5. If Primary Cluster UUID is NONE, then do not show that line at all.

Test Plan:
{F11973}

{F11974}

Created cluster and tested UI.
Attached screenshots

Reviewers: sid, karthik, bogdan, bharat

Reviewed By: bogdan, bharat

Subscribers: hector, ybase, bharat

Differential Revision: https://phabricator.dev.yugabyte.com/D6307
@m-iancu
Copy link
Contributor

m-iancu commented Mar 7, 2022

Fixed by 9093416.

@m-iancu m-iancu closed this as completed Mar 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
kind/enhancement This is an enhancement of an existing feature
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@m-iancu @kmuthukk @ndeodhar