Skip to content

feat(pre-aggregates): label required-filter misses - #27068

Draft
IrakliJani wants to merge 1 commit into
fix(pre-aggregates)/defer-required-filtersfrom
feat(pre-aggregates)/identify-required-filter-misses
Draft

feat(pre-aggregates): label required-filter misses#27068
IrakliJani wants to merge 1 commit into
fix(pre-aggregates)/defer-required-filtersfrom
feat(pre-aggregates)/identify-required-filter-misses

Conversation

@IrakliJani

@IrakliJani IrakliJani commented Aug 9, 2026

Copy link
Copy Markdown
Contributor

Related: ZAP-769

Adds a dedicated miss reason for required-filter dimensions that are unavailable in a pre-aggregate. Keeps ordinary query-filter misses distinct and preserves the missing field ID for audit surfaces.

@github-actions

github-actions Bot commented Aug 9, 2026

Copy link
Copy Markdown

🧪 Test Selection

✅ Tests that will run

Test Description
Preview Environment Deploys a preview environment for testing
Frontend E2E Tests Runs Cypress app tests
Backend API Tests Runs Vitest API tests
CLI Tests Runs CLI integration and dbt version tests

⏭️ Tests skipped (no relevant file changes detected)

Test How to trigger manually
Timezone Tests Add test-timezone to PR description

Tip: Add test-all to your PR description to run all tests.

@github-actions

github-actions Bot commented Aug 9, 2026

Copy link
Copy Markdown

🛡️ Upgrade safety for self-hosted customers

Comparing against fix(pre-aggregates)/defer-required-filters (65392c1ac8).

  • Safe to upgrade normally. No downtime needed.
  • No database changes in this release.

What we looked at

What Result
Database changes none
REST API no breaking changes
MCP tools no breaking changes
Upgrade notes none
Technical details (raw JSON)
{
  "schemaVersion": "1",
  "version": "pr-27068",
  "previousVersion": "65392c1ac8",
  "releaseDate": "2026-08-09T20:51:40.962Z",
  "capabilities": [
    "migrations",
    "rest",
    "mcp",
    "upgrade"
  ],
  "migrations": {
    "present": false,
    "count": 0,
    "files": [],
    "ee": false
  },
  "compatibility": {
    "rollingUpdateSafe": true,
    "recommendedStrategy": "RollingUpdate",
    "notes": "No database migrations detected in this release. This marker only reflects the checks listed in `capabilities`. It does NOT detect code-only or config-only breaking changes (env defaults, removed Helm values, serialization/protocol changes), which can also break old pods during a rolling update."
  },
  "api": {
    "rest": {
      "checked": true,
      "breaking": false,
      "changes": []
    },
    "mcp": {
      "checked": true,
      "breaking": false,
      "changes": []
    }
  },
  "upgrade": {
    "minPreviousVersion": null,
    "requiredStop": false,
    "note": null,
    "sourceVersion": null,
    "kind": null,
    "requiredStops": []
  }
}

Automated upgrade-safety check. Once merged, it ships a small release-safety.json with the release so customers’ upgrade automation can read it. It covers database and API changes; it doesn’t yet catch config/env-var or data-format changes.

@github-actions

github-actions Bot commented Aug 9, 2026

Copy link
Copy Markdown

Your preview environment pr-27068 has been deployed.

Preview environment endpoints are available at:

@github-actions

github-actions Bot commented Aug 9, 2026

Copy link
Copy Markdown

Preview Environment

🌐 URL: https://lightdash-preview-pr-27068.lightdash.okteto.dev

📋 Logs: View in GCP Console

🔧 SSH: ./scripts/okteto-ssh.sh 27068

@IrakliJani
IrakliJani force-pushed the feat(pre-aggregates)/identify-required-filter-misses branch from ebf4dbf to 2945a29 Compare August 9, 2026 17:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants