kevinjqliu opened a new issue, #2064: URL: https://github.com/apache/iceberg-python/issues/2064
### Apache Iceberg version None ### Please describe the bug 🐞 Context https://github.com/apache/iceberg-python/pull/2013#discussion_r2122682998 Because `Table.properties` returns the table metadata's properties field https://github.com/apache/iceberg-python/blob/a67c5592f3243d255519581fedfcc5d93274b9c8/pyiceberg/table/__init__.py#L1131-L1134 adding/removing the table properties is modifying the table metadata's properties field. This is not the recommended usage of the [table metadata's properties field](https://iceberg.apache.org/spec/#table-metadata-fields) ``` properties A string to string map of table properties. This is used to control settings that affect reading and writing and is not intended to be used for arbitrary metadata. For example, commit.retry.num-retries is used to control the number of commit retries. ``` Catalogs should handle figuring out where to properly save the table's properties. For example, Hive catalog should save the table's properties in the hive table parameters. ### Willingness to contribute - [ ] I can contribute a fix for this bug independently - [ ] I would be willing to contribute a fix for this bug with guidance from the Iceberg community - [ ] I cannot contribute a fix for this bug at this time -- 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: issues-unsubscr...@iceberg.apache.org.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
