D. Ferruzzi
D. Ferruzzi
Hi @tim-finnigan and @kdaily. We have run into this in Airflow as well, it's still an issue. Tasks are completing as expected, but the error message is getting printed to...
Hey @litalkat, we're likely going to need a little more info for this. Is there any stack trace with the error or any indication of a line number we can...
Thanks for getting this set up!
Had a look and I don't have anything extra to add; I like the direction this is going. I'd need to see some examples to have any real opinion on...
Yeah, I'm with @potiuk and @kaxil here. We already ran around in circles on this topic in the previous seconds/milliseconds metrics PR just a couple weeks ago. It's a mess,...
I'm fine with millis. My main problem is standardizing on the statsd and breaking everyone else. In fact, after the last Airflow 3 dev call where they were discussing dropping...
and see this one as well. https://github.com/apache/airflow/issues/40800 OTel will be the default starting in 3.0, StatsD will be included but secondary/backup/legacy. Given that context, Here's what I think: This can...
If I may, can you add some indication to the title that this is related to OTel Traces specifically? We also have OTel Metrics implemented and it would be nice...
I got most of the way though this, I'll finish reviewing tomorrow. Overall looks like a sound plan, very well documented and commented.
Alright, I apologize for all the individual comments, I wasn't able to carve out a big block of time to get it all in one go, so I made my...