sidshas03 commented on issue #55693: URL: https://github.com/apache/airflow/issues/55693#issuecomment-3301215206
Hi @techgnosis @jroachgolf84 Commit: sidshas03/airflow@`1a4eb3f8fe5956d238b7ec9453009ebf2bc19ca1` Link: [https://github.com/sidshas03/airflow/commit/1a4eb3f8fe5956d238b7ec9453009ebf2bc19ca1](https://github.com/sidshas03/airflow/commit/1a4eb3f8fe5956d238b7ec9453009ebf2bc19ca1) **What it does** * Adds **early per-tag validation (100 chars)** in the SDK DAG path * Raises `AirflowException` naming the offending tag (short preview) and its **actual length** * Adds a short docs note under core DAG concepts about the **per-tag** limit * Adds unit tests: **100 OK**, **101 fails**, **mixed tags fails** identifying the long tag **Notes** * No DB/schema changes (matches `dag_tag.name VARCHAR(100)`) * I’ll open a fresh branch off `apache/main` containing **only**: * `task-sdk/src/airflow/sdk/definitions/dag.py` * `tests/models/test_dag_tags.py` * `docs/apache-airflow/core-concepts/dags.rst` * If you prefer a different preview length or file placement, I can adjust. If this looks good, I’ll submit the clean PR with **“Resolves #55693.”** -- 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]
