Add support for 'overwrite' option in register_table#3290
Add support for 'overwrite' option in register_table#3290ebyhr wants to merge 1 commit intoapache:mainfrom
Conversation
|
Looks good to me, one small nit would be to add some tests in integration tests here if the iceberg rest image supports the overwriting flag |
|
The current test target (1.10.0) supports the option in OpenAPI yml, but the catalog doesn't support it. We can add an integration test once 1.11.0 (apache/iceberg#15525) is released. |
ebyhr
changed the title
|
any chance for this to get merged, currently, i am doing tuncate and add files with other tool, and it is a pain as it is not atomic |
| Args: | ||
| identifier (Union[str, Identifier]): Table identifier for the table | ||
| metadata_location (str): The location to the metadata | ||
| overwrite (bool): Whether to overwrite the existing table, default False |
There was a problem hiding this comment.
should we add NotImplementedError here too?
There was a problem hiding this comment.
This method in dynamodb and noop always raises NotImplementedError. Do we still need a separate logc for the overwrite argument?
| Args: | ||
| identifier (Union[str, Identifier]): Table identifier for the table | ||
| metadata_location (str): The location to the metadata | ||
| overwrite (bool): Whether to overwrite the existing table, default False |
There was a problem hiding this comment.
nit: raise NotImplementedError here too?
5 participants
Rationale for this change
Are these changes tested?
Are there any user-facing changes?
This PR adds
overwriteparameter tocatalog.register_tablemethod.