Add expire snapshots to cli - #3379
Conversation
|
@kevinjqliu @geruh @Fokko ptal! I think this will be really useful for users. |
| spec Return the partition spec of the table. | ||
| uuid Return the UUID of the table. | ||
| version Print pyiceberg version. | ||
| create Operation to create a namespace. |
There was a problem hiding this comment.
If we add all of the maintenance commands, this will increase the number of commands.
We could have a maintenance group, where the command would be pyiceberg maintenance expire-snapshots
click has support for this, but I find it pretty cumbersome. It also makes it harder for LLMs to discover the new capability.
32c69a1 to
2512a5c
Compare
2512a5c to
9d6d7fb
Compare
|
This pull request has been marked as stale due to 30 days of inactivity. It will be closed in 1 week if no further activity occurs. If you think that's incorrect or this pull request requires a review, please simply write any comment. If closed, you can revive the PR at any time and @mention a reviewer or discuss it on the dev@iceberg.apache.org list. Thank you for your contributions. |
2 participants
Rationale for this change
We should expose out the maintenance operations on the CLI! It's really powerful to be able to use the CLI to run these commands rather than spin up a full Spark cluster (or even write a Python script)
Are these changes tested?
I included some tests against the InMemoryCatalog. The CLI is a small shim against the actual operation.
Are there any user-facing changes?