ferruzzi commented on PR #44346: URL: https://github.com/apache/airflow/pull/44346#issuecomment-2498665811
Interesting. I wonder if it might be safer to add some code which looks for that envvar and keeps json as the default, add a deprecation note that this will change in the future, and after the deprecation period remove the default and let the envvar and OTel default behavior do its thing. That would let you get the functionality you need without breaking anyone else without notice. But I also do not have any statistics on how many people might be affected either way so, it's hard to make a call if this is a "bugfix" we can fix immediately or a "new breaking feature" we need to warn about. -- 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]
