Skip to content

Add view operations to CLI - #3926

Open
ebyhr wants to merge 1 commit into
apache:mainfrom
ebyhr:ebi/cli-view
Open

Add view operations to CLI#3926
ebyhr wants to merge 1 commit into
apache:mainfrom
ebyhr:ebi/cli-view

Conversation

@ebyhr

@ebyhr ebyhr commented Sep 10, 2026

Copy link
Copy Markdown
Member

Rationale for this change

The PyIceberg CLI (pyiceberg command) supports inspecting and managing tables and namespaces, but has no support for views. As Iceberg view support has been added to catalogs, users need equivalent CLI operations to discover and inspect views.

Are these changes tested?

Yes

Are there any user-facing changes?

Yes

Dependency

Depends on #3934, which contains the generic describe entity-detection behavior. Merge #3934 first, then rebase this PR onto main.

@kevinjqliu
kevinjqliu changed the base branch from main to kevinjqliu-cli-describe-entity-detection September 11, 2026 17:44
@kevinjqliu
kevinjqliu force-pushed the ebi/cli-view branch 2 times, most recently from 350641b to d1af1c3 Compare September 11, 2026 17:53
@kevinjqliu

kevinjqliu commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

Hi @ebyhr sorry for the noise on this PR. Im running a new workflow with copilot, still getting use to it.

I made an attempt to refactor how describe handle any in #3934 and rebased this PR on top. Do you mind taking a look at both PRs? 😄

@kevinjqliu
kevinjqliu requested review from Fokko and geruh September 11, 2026 18:08

@kevinjqliu kevinjqliu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM, I gave it another review.

I'm going to see what @ebyhr has to say 😄

@kevinjqliu
kevinjqliu deleted the branch apache:main September 12, 2026 02:23
@kevinjqliu kevinjqliu closed this Sep 12, 2026
@kevinjqliu kevinjqliu reopened this Sep 12, 2026
@kevinjqliu
kevinjqliu changed the base branch from kevinjqliu-cli-describe-entity-detection to main September 12, 2026 02:27
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
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.

2 participants