villebro opened a new pull request, #43846: URL: https://github.com/apache/superset/pull/43846
### SUMMARY Addresses review feedback on #43407 ([discussion](https://github.com/apache/superset/pull/43407#discussion_r3930062725)): enabling Global Async Queries floors async chart-result cache TTLs (default 5 min), which can override a shorter operator retention policy and increase Redis usage, but the operator docs didn't mention or explain how to tune it. Adds an **"Async Query Result Cache TTL"** section to `docs/admin_docs/configuration/cache.mdx` covering: - **Why the floor exists (motivation):** an async `/chart/data` request runs on a background task that writes the result to the data cache, and the browser then re-issues the request to read it back. If the effective TTL is shorter than the async round trip (task execution + poll interval + re-fetch), the entry can be evicted before the client reads it, leaving the chart stuck re-running. - **How it interacts with the existing timeout chain:** async-only (sync requests unaffected even with GAQ on), raises only TTLs shorter than the floor, and leaves `0` ("cache forever") untouched. - **How to tune `GLOBAL_ASYNC_QUERIES_MIN_CACHE_TTL`** and the retention / Redis-usage trade-off (don't lower it below your worst-case async round trip). ### BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF Docs-only. Preview: the new section renders under Configuration → Caching, after "Chart Cache Timeout". ### TESTING INSTRUCTIONS Docs-only change; no code paths affected. Verify the section renders in the docs preview and the `#feature-flags` anchor resolves. ### ADDITIONAL INFORMATION - [ ] Has associated issue: - [ ] Required feature flags: - [ ] Changes UI - [ ] Includes DB Migration (follow approval process in [SIP-59](https://github.com/apache/superset/issues/13351)) - [ ] Migration is atomic, supports rollback & is backwards-compatible - [ ] Confirm DB migration upgrade and downgrade tested - [ ] Runtime estimates and downtime expectations provided - [ ] Introduces new feature or API - [ ] Removes existing feature or API -- 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]
