howardyoo commented on PR #40802: URL: https://github.com/apache/airflow/pull/40802#issuecomment-2471659171
Yes, I actually thought about that as well! But for now, we may just have to make decision on perhaps truncating, because it would actually require less work and thus more robust.(Maybe in the future we may want to make it as a configurable option of having truncate by head, tail, or head+tail.)Yeah traces and metrics should be treated differently and we should try hard to reduce the confusion and noise around that.Sent from my iPhoneOn Nov 12, 2024, at 1:19 PM, D. Ferruzzi ***@***.***> wrote: I like the idea of truncating the logs. The big question there would be if we should head the logs or tail the logs.... if we are cutting them short, is it more useful to have the beginning or the end? I'd say maybe the end, but honestly, it's so hard to say. Many times you have to scroll back to find a real root cause. Either way, if I may humbly make a request, can we please make an effort to refer to Traces and Metrics rather than calling both OTel? They are two distinct features which both use OTel, but I feel like the recent discussion around improving Traces has undermined confidence in the OTel Metrics implementation which AFAIK is not having any issues. There is a project underway to improve the Metrics docs which seems to be stalled because of confusion around the Traces discussion. The two are not related and IMHO discussion around Traces shouldn't be affecting the Metrics improvement project. —Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: ***@***.***> -- 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]
