Skip to content

feat(cli): add --prefix option for REST catalog#3061

Merged
Fokko merged 1 commit into
apache:mainfrom
mrutunjay-kinagi:fix-3059-cli-prefix
Feb 18, 2026
Merged

feat(cli): add --prefix option for REST catalog#3061
Fokko merged 1 commit into
apache:mainfrom
mrutunjay-kinagi:fix-3059-cli-prefix

Conversation

@mrutunjay-kinagi

@mrutunjay-kinagi mrutunjay-kinagi commented Feb 17, 2026

Copy link
Copy Markdown
Contributor

Rationale for this change

The CLI already supports --uri and --credential, but REST catalogs that require {prefix} could only be configured via environment/config file. This change adds a first-class --prefix CLI option and forwards it to load_catalog.

Are these changes tested?

Yes.

  • Added test_prefix_cli_option_forwarded_to_catalog in tests/cli/test_console.py.
  • Verified with targeted pytest run.

Are there any user-facing changes?

Yes.

  • New CLI option: --prefix.
  • Updated CLI docs in mkdocs/docs/cli.md.

@DerGut DerGut left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I tested it in our setup and it works as expected 👍

@mrutunjay-kinagi

mrutunjay-kinagi commented Feb 18, 2026

Copy link
Copy Markdown
Contributor Author

I tested it in our setup and it works as expected 👍

Thank you so much !I've to wait for this to be approved.

@Fokko Fokko 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.

Nice addition, thanks @mrutunjay-kinagi for working on this and thanks @DerGut for the review 🚀

@Fokko Fokko merged commit 9c9f160 into apache:main Feb 18, 2026
12 checks passed
@mrutunjay-kinagi

Copy link
Copy Markdown
Contributor Author

Nice addition, thanks @mrutunjay-kinagi for working on this and thanks @DerGut for the review 🚀

Thank you @Fokko !

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.

3 participants