szehon-ho commented on PR #18049: URL: https://github.com/apache/iceberg/pull/18049#issuecomment-5825988123
Thanks for addressing the collision issue. Looking at this again, `Table.properties()` is also used by Spark’s `SHOW CREATE TABLE`, so catalog-provided labels would be emitted as persisted `TBLPROPERTIES`. I’m working on fixing this on the Spark side by separating display-only properties: https://github.com/apache/spark/pull/59036. If accepted, Iceberg can expose labels through that API for supported Spark versions without affecting generated DDL. -- 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]
