kbendick commented on PR #4645: URL: https://github.com/apache/iceberg/pull/4645#issuecomment-1115301314
> This is actually a table property, should it add a description here [https://iceberg.apache.org/docs/latest/configuration/](url) ? @felixYyu @kbendick Appreciate the consideration for looking after the main table configurations, but I don't think this merits being listed in the list of all configurations. That's primarily for Iceberg-specific configuration properties. Table comment being a table property is more of a detail of Spark (and other engines). So it's probably best left for them to document. Ideally, that main `configurations` page should be a place where users can look up Iceberg-specific table properties, so I think that just having the example is sufficient. -- 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]
