Skip to content

docs(cache): document NATIVE_FILTER_OPTIONS_CACHE_TIMEOUT - #43592

Merged
rusackas merged 1 commit into
masterfrom
fix/from-pr-38910-needs-docs-fix-native-filters-use-filter
Aug 27, 2026
Merged

docs(cache): document NATIVE_FILTER_OPTIONS_CACHE_TIMEOUT#43592
rusackas merged 1 commit into
masterfrom
fix/from-pr-38910-needs-docs-fix-native-filters-use-filter

Conversation

@rusackas

Copy link
Copy Markdown
Member

SUMMARY

#38910 added NATIVE_FILTER_OPTIONS_CACHE_TIMEOUT to superset/config.py so native filter dropdown queries can use their own cache TTL instead of DATA_CACHE_CONFIG, but docs/admin_docs/configuration/cache.mdx never documented this setting in the main "Chart Cache Timeout" section (it's only mentioned in passing under the later cache warm-up section). This adds a short paragraph next to DATA_CACHE_CONFIG explaining the setting, its valid values (None/-1/0/positive int), and where it sits in the timeout precedence chain.

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

N/A (docs only)

TESTING INSTRUCTIONS

Render docs/admin_docs/configuration/cache.mdx and confirm the new "Chart Cache Timeout" section content is accurate and matches QueryContextProcessor.get_cache_timeout().

ADDITIONAL INFORMATION

  • Has associated issue:
  • Required feature flags:
  • Changes UI
  • Includes DB Migration
  • Introduces new feature or API
  • Removes existing feature or API

Follow-up docs fix identified while triaging #38910.

…hart cache timeout

The general Flask-Caching backend section didn't mention the dedicated
timeout knob for native filter option queries introduced in #38910;
document it next to DATA_CACHE_CONFIG in the Chart Cache Timeout section.

Co-Authored-By: Claude <noreply@anthropic.com>
@dosubot dosubot Bot added doc:user User / Superset documentation infra:caching Infra setup and configuration related to caching labels Aug 27, 2026
@bito-code-review

bito-code-review Bot commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

Code Review Agent Run #9b7915

Actionable Suggestions - 0
Review Details
  • Files reviewed - 1 · Commit Range: 930fad4..930fad4
    • docs/admin_docs/configuration/cache.mdx
  • Files skipped - 0
  • Tools
    • Whispers (Secret Scanner) - ✔︎ Successful
    • Detect-secrets (Secret Scanner) - ✔︎ Successful

Bito Usage Guide

Commands

Type the following command in the pull request comment and save the comment.

  • /review - Manually triggers an incremental AI Review.

  • /review full - Manually triggers a full AI Review.

  • /pause - Pauses automatic reviews on this pull request.

  • /resume - Resumes automatic reviews.

  • /resolve - Marks all Bito-posted review comments as resolved.

  • /abort - Cancels all in-progress reviews.

Refer to the documentation for additional commands.

Configuration

This repository uses Superset You can customize the agent settings here or contact your Bito workspace admin at evan@preset.io.

Documentation & Help

AI Code Review powered by Bito Logo

@github-actions github-actions Bot added doc Namespace | Anything related to documentation preset-io labels Aug 27, 2026
@netlify

netlify Bot commented Aug 27, 2026

Copy link
Copy Markdown

Deploy Preview for superset-docs-preview ready!

Name Link
🔨 Latest commit 930fad4
🔍 Latest deploy log https://app.netlify.com/projects/superset-docs-preview/deploys/6a905d0f0023ff0008e43837
😎 Deploy Preview https://deploy-preview-43592--superset-docs-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@rusackas
rusackas merged commit 5c3c636 into master Aug 27, 2026
76 checks passed
@rusackas
rusackas deleted the fix/from-pr-38910-needs-docs-fix-native-filters-use-filter branch August 27, 2026 19:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc:user User / Superset documentation doc Namespace | Anything related to documentation infra:caching Infra setup and configuration related to caching preset-io size/S

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants