Skip to content

(via govambam): Fix cache key collision between dbt_ls and yaml_selectors Airflow Variab#2

Merged
macroscopeapp[bot] merged 1 commit into
review-pr-2261from
macroscope/review-pr-2261
Feb 25, 2026
Merged

(via govambam): Fix cache key collision between dbt_ls and yaml_selectors Airflow Variab#2
macroscopeapp[bot] merged 1 commit into
review-pr-2261from
macroscope/review-pr-2261

Conversation

@macroscopeapp
Copy link
Copy Markdown

@macroscopeapp macroscopeapp Bot commented Feb 25, 2026

Macroscope: Fix It For Me

Activity

Currently: Auto-merged 71e7cdf

Previously

Note

Prevent cache key collision by suffixing Airflow Variable keys for dbt ls and YAML selectors in DbtGraph methods in graph.py

Update DbtGraph to use "{cache_key}_dbt_ls" for dbt ls cache and "{cache_key}_yaml_selectors" for YAML selectors cache when reading and writing via Airflow Variables in graph.py.

📍Where to Start

Start with the cache read/write logic in DbtGraph within graph.py, focusing on the methods handling Airflow Variable keys for dbt ls and YAML selector caches.

Macroscope summarized fc04e1f.

@macroscopeapp macroscopeapp Bot merged commit 71e7cdf into review-pr-2261 Feb 25, 2026
1 check passed
@macroscopeapp macroscopeapp Bot deleted the macroscope/review-pr-2261 branch February 25, 2026 23:17
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.

1 participant