rdblue commented on code in PR #14440:
URL: https://github.com/apache/iceberg/pull/14440#discussion_r3364802629
##########
docs/docs/catalog-properties.md:
##########
@@ -33,6 +33,7 @@ Iceberg catalogs support using catalog properties to
configure catalog behaviors
| clients | 2 | client pool size
|
| cache-enabled | true | Whether to cache
catalog entries |
| cache.expiration-interval-ms | 30000 | How long catalog
entries are locally cached, in milliseconds; 0 disables caching, negative
values disable expiration |
+ cache.expire-after-write-ms | 0 | Duration in
milliseconds to expire a table from the cache after being created.tables will
not refresh on write. 0 disables this policy. its disabled by default |
Review Comment:
This is long and its default is confusing (both disabled and 0). Also please
use sentence case and standard spacing (one space between sentences after `.`).
--
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]
