dstandish commented on PR #27787:
URL: https://github.com/apache/airflow/pull/27787#issuecomment-1329464951

   > I think w do not have anything else to do but simply implement Open 
Telemetry.
   > 
   > It's already been discussed, and approved. We just need to implement it 
https://cwiki.apache.org/confluence/display/AIRFLOW/AIP-49+OpenTelemetry+Support+for+Apache+Airflow
   > 
   > In the form defined and researched by @howardyoo - this is the best way we 
can implement "smarts" in the telemetry of Airflow.
   
   I looked at the AIP, but it looks like logging was deliberately excluded 
from it:
   
   > While Metrics seems to be ready to integrate within the scope of this AIP 
and Open-Telemetry integrations (no surprises foreseen), we believe Log 
integration requires more work and POC that should be accompanied with an 
approach to rewrite and simplify logging integration as Airflow logging is 
particularly vulnerable to disruption and it is integrated with Airflow UI in a 
subtle way. We propose to exclude Logging from the scope of the AIP (and 
implement it as a follow-up AIP).
   
   


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