sidshas03 commented on issue #55693: URL: https://github.com/apache/airflow/issues/55693#issuecomment-3294472251
Dear @techgnosis, I am interested on taking this PR. Here is what I'm planning to complete this request. ### Problem Currently Airflow has a hard limit of 100 characters for combined length of all DAG tags. This is quite restrictive and causes issues when users want to add descriptive tags to their DAGs. When this limit is exceeded, the DAG fails to load without any clear error message. ### What I'm planning to do 1. **Increase the character limit** from 100 to 500 characters for combined DAG tags 2. **Add proper validation** with clear error messages when limit is exceeded 3. **Update documentation** to mention this limitation 4. **Add unit tests** to verify the new limit works correctly ### Approach - First, I'll find where this 100-character limit is enforced in the codebase - Update the validation logic to use 500 characters instead - Add better error messages that tell users exactly what went wrong - Write tests to ensure the new limit works properly - Update any relevant documentation ### Files I expect to modify - DAG validation logic (probably in `airflow/models/dag.py`) - Error handling for tag validation - Unit tests for DAG tag validation - Documentation files ### Testing plan - Create test DAGs with tags totaling exactly 500 characters (should work) - Create test DAGs with tags totaling 501+ characters (should fail with clear error) - Verify existing functionality still works - Test edge cases This will give users much more flexibility in tagging their DAGs while still maintaining reasonable limits. Thank you Siddharthan P S -- 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]
