bujjibabukatta opened a new pull request, #68124:
URL: https://github.com/apache/airflow/pull/68124

   Fixes #68018
   
   Addresses all points raised in the previous review:
   
   - Added missing `from ..exceptions import InvalidStatsNameException` so the 
`except` clause in `name_is_otel_safe()` correctly catches the exception 
instead of raising `NameError`
   - Added `TestNameIsOtelSafe` regression tests — non-ASCII names return 
`False` without raising, valid ASCII names still return `True`
   - Imported `name_is_otel_safe` in the test file so the new tests can run


-- 
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]

Reply via email to