Fokko commented on issue #123:
URL: https://github.com/apache/iceberg-python/issues/123#issuecomment-1791855246
Thanks for raising this @pdames
In hindsight, I think adding the catalog name to the identifier was a bad
choice. We tried to mimic the behavior of Java, but I don't see any advantages
of having this (since we have a reference to the catalog anyway).
If you want to refresh a table, you could also run:
```python
table = catalog.load_table(("test_namespace", "test_table")) # tuple
identifier also works...
table.refresh()
```
However, I still think that your example should work as well.
Removing it will break existing behavior, so I think @danielcweeks's
suggestion is best. We probably also want to fix this for the
renaming/delete/etc scenario:
```python
catalog.rename_table(table.identifier, 'database.new_table_name')
catalog.drop_table(table.identifier)
catalog.purge_table(table.identifier)
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
