Skip to content

lakebox: always show NAME column in lakebox list#5382

Merged
akshaysingla-db merged 1 commit into
databricks:demo-lakeboxfrom
akshaysingla-db:akshay/lakebox-list-name-always
May 29, 2026
Merged

lakebox: always show NAME column in lakebox list#5382
akshaysingla-db merged 1 commit into
databricks:demo-lakeboxfrom
akshaysingla-db:akshay/lakebox-list-name-always

Conversation

@akshaysingla-db
Copy link
Copy Markdown
Collaborator

Yunquan flagged on the bug-bash form: the table shape changed between calls — the NAME column appeared the moment any sandbox had a custom --name set, and vanished when none did. Scripts that parsed list output had to handle two column layouts; users had to mentally remap columns based on workspace state.

Always render the column. Sandboxes without a custom name (Name == "" or Name == SandboxID) display - (faint), same convention we already use elsewhere. The column width still scales to the longest actual name, so workspaces with only unnamed sandboxes render a narrow NAME column of dashes — visually quiet but structurally present.

Co-authored-by: Isaac

Changes

Why

Tests

Yunquan flagged on the bug-bash form: the table shape changed between
calls — the NAME column appeared the moment any sandbox had a custom
`--name` set, and vanished when none did. Scripts that parsed `list`
output had to handle two column layouts; users had to mentally remap
columns based on workspace state.

Always render the column. Sandboxes without a custom name (Name == ""
or Name == SandboxID) display `-` (faint), same convention we already
use elsewhere. The column width still scales to the longest *actual*
name, so workspaces with only unnamed sandboxes render a narrow
NAME column of dashes — visually quiet but structurally present.

Co-authored-by: Isaac
@akshaysingla-db akshaysingla-db merged commit e72597d into databricks:demo-lakebox May 29, 2026
5 of 6 checks passed
@github-actions
Copy link
Copy Markdown
Contributor

Waiting for approval

Based on git history, these people are best suited to review:

  • @shuochen0311 -- recent work in cmd/lakebox/
  • @pietern -- recent work in cmd/lakebox/

Eligible reviewers: @andrewnester, @anton-107, @denik, @renaudhartert-db, @shreyas-goenka, @simonfaltum

Suggestions based on git history. See OWNERS for ownership rules.

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.

1 participant