FANNG1 commented on code in PR #8980:
URL: https://github.com/apache/gravitino/pull/8980#discussion_r2618187368
##########
docs/iceberg-rest-service.md:
##########
@@ -447,6 +448,20 @@ Gravitino features a pluggable cache system for updating
or retrieving table met
Gravitino provides the build-in
`org.apache.gravitino.iceberg.common.cache.LocalTableMetadataCache` to store
the cached data in the memory. You could also implement your custom table
metadata cache by implementing the
`org.apache.gravitino.iceberg.common.cache.TableMetadataCache` interface.
+### Iceberg scan plan cache configuration
+
+Gravitino caches scan plan results to speed up repeated queries with identical
parameters. The cache uses snapshot ID as part of the cache key, so queries
against different snapshots will not use stale cached data.
+
+| Configuration item | Description
| Default value | Required | Since
Version |
+|------------------------------------------------------------|----------------------------------------------------------|---------------|----------|---------------|
+| `gravitino.iceberg-rest.scan-plan-cache-impl` | The
implementation of the scan plan cache. | (none) | No
| 1.1.0 |
+| `gravitino.iceberg-rest.scan-plan-cache-capacity` | The capacity of
the scan plan cache. | 200 | No | 1.1.0
|
+| `gravitino.iceberg-rest.scan-plan-cache-expire-minutes` | The expiration
time (in minutes) of the scan plan cache. | 60 | No | 1.1.0
|
+
+The scan plan cache uses snapshot ID as part of the cache key, ensuring
automatic invalidation when table data changes. This can provide significant
speedup for repeated queries like dashboard refreshes or BI tool queries.
+
+Gravitino provides the built-in
`org.apache.gravitino.iceberg.service.cache.LocalScanPlanCache` to store the
cached data in memory. If no implementation is specified, this default
implementation will be used automatically. You can also implement your custom
scan plan cache by implementing the
`org.apache.gravitino.iceberg.service.cache.ScanPlanCache` interface.
Review Comment:
Could you update the doc about default implementation?
--
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]