Skip to content

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

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

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

Conversation

@mrutunjay-kinagi
Copy link
Contributor

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

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.

Copy link

@DerGut DerGut left a comment

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
Copy link
Contributor Author

mrutunjay-kinagi commented Feb 18, 2026

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

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

Copy link
Contributor

@Fokko Fokko left a comment

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
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