Skip to content

Implement rename view to the rest catalog - #3712

Closed
nhuantho wants to merge 1 commit into
apache:mainfrom
nhuantho:rest_catalog/add-rename_view
Closed

Implement rename view to the rest catalog#3712
nhuantho wants to merge 1 commit into
apache:mainfrom
nhuantho:rest_catalog/add-rename_view

Conversation

@nhuantho

Copy link
Copy Markdown

Rationale for this change

I want to implement rename view to the reset catalog.

Are these changes tested?

I added

  • test_rename_view_200
  • test_rename_view_from_self_identifier_200
  • test_rename_view_source_namespace_does_not_exist
  • test_rename_view_destination_namespace_does_not_exist

Are there any user-facing changes?

No

@nhuantho

Copy link
Copy Markdown
Author

@Fokko could you please review for me?

@ebyhr

ebyhr commented Jul 27, 2026

Copy link
Copy Markdown
Member

We already have an existing PR #2149.

@nhuantho

Copy link
Copy Markdown
Author

@ebyhr thanks for your review. I didn't check whether the PRs existed, so I closed this mr.

@nhuantho nhuantho closed this Jul 27, 2026
@rambleraptor

Copy link
Copy Markdown
Collaborator

@nhuantho thanks for the PR! Could you comment / review #2149? It should be hopefully in a good place to get merged shortly.

@nhuantho

Copy link
Copy Markdown
Author

@rambleraptor sure, I will review your pr

@nhuantho
nhuantho deleted the rest_catalog/add-rename_view branch July 27, 2026 18:34
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