vatsrahul1001 opened a new pull request, #74390: URL: https://github.com/apache/airflow/pull/74390
The mapped-task-group skip-decision query added in #74283 reads `XComModel.task_id` / `XComModel.map_index` / `XComModel.value` directly in `with_only_columns(...)`. The `check-xcom-model-columns` prek hook forbids reading `XComModel.<column>` directly in a SQLAlchemy query (it must go through `xcom_entity(query).<column>`), so the repo-wide Static checks job is now red on `main` and on every PR cut from it. This reads the three columns through `xcom_entity(query)` instead, matching the hook and the pattern used elsewhere (e.g. `models/taskinstance.py`, `serialization/definitions/operatorlink.py`). No behavior change. Verified `scripts/ci/prek/check_xcom_model_columns.py` passes on the file. related: #74283 --- ##### Was generative AI tooling used to co-author this PR? - [X] Yes — Claude Code (Opus 4.8) Generated-by: Claude Code (Opus 4.8) following [the guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions) -- 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]
