Skip to content

ui: add transactions page to Admin UI#54261

Merged
craig[bot] merged 1 commit intocockroachdb:masterfrom
vladlos:transactions-page
Sep 24, 2020
Merged

ui: add transactions page to Admin UI#54261
craig[bot] merged 1 commit intocockroachdb:masterfrom
vladlos:transactions-page

Conversation

@vladlos
Copy link
Copy Markdown
Contributor

@vladlos vladlos commented Sep 11, 2020

This commit adds the new Transactions Page to the Admin UI.

This page is much like the Statements Page but differs in that
it shows Transaction-level statistics in tabular and detail form.

Every Transaction is able to display its contained Statements
for more detailed analysis.

The page and its components are imported from the
admin-ui-components library.

Depends on: cockroachdb/yarn-vendored#38

Release justification: low-risk high impact addition to Admin UI

Release note (admin ui change): add Transactions and Transactions
details pages. These pages allow for viewing stats at the
transaction level.

@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

@dhartunian dhartunian changed the title transactions page wip ui: Add Transactions Page to Admin UI Sep 14, 2020
@dhartunian dhartunian changed the title ui: Add Transactions Page to Admin UI ui: add transactions page to Admin UI Sep 14, 2020
@dhartunian dhartunian marked this pull request as ready for review September 14, 2020 14:40
@dhartunian dhartunian requested a review from a team September 14, 2020 14:40
This commit adds the new Transactions Page to the Admin UI.

This page is much like the Statements Page but differs in that
it shows Transaction-level statistics in tabular and detail form.

Every Transaction is able to display its contained Statements
for more detailed analysis.

The page and its components are imported from the
`admin-ui-components` library.

Release justification: low-risk high impact addition to Admin UI

Release note (admin ui change): add Transactions and Transactions
details pages. These pages allow for viewing stats at the
transaction level.
Copy link
Copy Markdown
Contributor

@nathanstilwell nathanstilwell left a comment

Choose a reason for hiding this comment

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

LGTM

@dhartunian
Copy link
Copy Markdown
Collaborator

bors r+

@craig
Copy link
Copy Markdown
Contributor

craig bot commented Sep 24, 2020

Build succeeded:

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.

4 participants