Add rename_view to REST Catalog#2149
Conversation
|
@Fokko @kevinjqliu mind taking a look when you can? thanks! |
rambleraptor
force-pushed
the
rename_view
branch
from
f4e2a61 to
756ee0b
Compare
rambleraptor
force-pushed
the
rename_view
branch
from
756ee0b to
8ad2e55
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. |
rambleraptor
force-pushed
the
rename_view
branch
from
8ad2e55 to
9188ced
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. |
rambleraptor
force-pushed
the
rename_view
branch
from
c8996c2 to
7f39e33
Compare
|
@geruh looks like you've been going through the View PRs. Mind taking a look at this one? Follows the others near-identically. |
|
@ebyhr can you take a look? You've been doing a lot of work around views lately. |
|
@kevinjqliu @Fokko @geruh mind taking a look? This will help with all of the View work that's been done. |
Co-authored-by: Kevin Liu <kevinjqliu@users.noreply.github.com>
abnobdoss
left a comment
abnobdoss
left a comment
There was a problem hiding this comment.
Looks good! The only concern I have is the point you highlighted in the PR description of the discrepancy between rename_table returning a Table but rename_view returning None. It sounds like the right fix would be to change rename_table to return None to be aligned to the spec. Since that is a breaking change I imagine we likely would have to wait till 1.0.0 to align those functions.
|
I agree that |
5 participants
Rationale for this change
As part of #818, this adds
rename_viewsupport to the Iceberg REST Catalog.The REST Catalog spec specifies that both
rename_viewandrename_tabledo not return anything. Currently,rename_tablereturns the table. I'm happy to change that API if we want, but it would technically be a breaking change.Are these changes tested?
Added unit tests.
Are there any user-facing changes?
rename_viewsupport to Iceberg REST Catalog.