Reviewed all commit messages.
Reviewable status: complete! 0 of 0 LGTMs obtained (waiting on @koorosh and @maryliag)

-- commits, line 4 at r1:
nit: s/cluster/clusters

pkg/ui/workspaces/cluster-ui/src/sortedtable/sortedtable.tsx, line 59 at r1 (raw file):

  // If true, the user can't overwrite the setting for this column. False if not defined.
  alwaysShow?: boolean;
  // If true, it will hide this column if it's tenant cluster. False is not defined.

nit: "If true, hide this column for tenant clusters. False if not defined."

pkg/ui/workspaces/cluster-ui/src/statementsPage/statementsPage.tsx, line 351 at r1 (raw file):

        // list if the list is not empty.
        // If the cluster is a tenant cluster we don't care
        // about regions

nit: trailing period.

pkg/ui/workspaces/cluster-ui/src/statementsPage/statementsPage.tsx, line 368 at r1 (raw file):

        // list if the list is not empty.
        // If the cluster is a tenant cluster we don't care
        // about nodes

nit: trailing period.

pkg/ui/workspaces/cluster-ui/src/statementsPage/statementsPage.tsx, line 404 at r1 (raw file):

    const isEmptySearchResults = statements?.length > 0 && search?.length > 0;
    // If the cluster is a tenant cluster we don't show info
    // about nodes/regions

nit: trailing period.

pkg/ui/workspaces/cluster-ui/src/statementsPage/statementsPage.tsx, line 482 at r1 (raw file):

              showScan={true}
              showRegions={regions.length > 1 && !isTenant}
              showNodes={nodes.length > 1 && !isTenant}

Are these clauses redundant with setting nodes and regions to empty arrays above?

pkg/ui/workspaces/cluster-ui/src/transactionsPage/transactionsPage.tsx, line 237 at r1 (raw file):

            );
            // If the cluster is a tenant cluster we don't show info
            // about nodes/regions

nit: trailing period.

pkg/ui/workspaces/cluster-ui/src/transactionsPage/transactionsPage.tsx, line 295 at r1 (raw file):

                      filters={filters}
                      showRegions={regions.length > 1 && !isTenant}
                      showNodes={nodes.length > 1 && !isTenant}

Are these clauses redundant with setting nodes and regions to empty arrays above?

pkg/ui/workspaces/cluster-ui/src/transactionsPage/utils.ts, line 144 at r1 (raw file):

      if (regions.length == 0 && nodes.length == 0) return true;
      // If the cluster is a tenant cluster we don't care
      // about node/regions

nit: trailing period.

Reviewable status: complete! 0 of 0 LGTMs obtained (waiting on @koorosh and @matthewtodd)

pkg/ui/workspaces/cluster-ui/src/statementsPage/statementsPage.tsx, line 482 at r1 (raw file):

Good point, remove here and other cases.
Done

pkg/ui/workspaces/cluster-ui/src/transactionsPage/transactionsPage.tsx, line 295 at r1 (raw file):

Done.

Reviewable status: complete! 1 of 0 LGTMs obtained (waiting on @koorosh)