kevinjqliu commented on code in PR #14676: URL: https://github.com/apache/iceberg/pull/14676#discussion_r2558458181
########## docs/docs/configuration.md: ########## @@ -108,6 +108,7 @@ Iceberg tables support table properties to configure table behavior, like the de | history.expire.max-snapshot-age-ms | 432000000 (5 days) | Default max age of snapshots to keep on the table and all of its branches while expiring snapshots | | history.expire.min-snapshots-to-keep | 1 | Default min number of snapshots to keep on the table and all of its branches while expiring snapshots | | history.expire.max-ref-age-ms | `Long.MAX_VALUE` (forever) | For snapshot references except the `main` branch, default max age of snapshot references to keep while expiring snapshots. The `main` branch never expires. | +| gc.enabled | true | Disable garbage collection operations such as expiring snapshots or removing orphan files | Review Comment: ```suggestion | gc.enabled | true | Allows garbage collection operations such as expiring snapshots and removing orphan files | ``` nit: using "Disable" here is confusing, esp the property name has "enabled" and defaults to true -- 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]
