Christos Bisias
Christos Bisias
@ferruzzi I adjusted the title. The only change in this patch that is related to metrics, it's https://github.com/apache/airflow/pull/43941/files#diff-1cca954ec0be1aaf2c212e718c004cb0902a96ac60043bf0c97a782dee52cc32R85-R86 If you think that it's out of scope, then I can remove...
My initial approach wasn't considering scheduler HA. I've updated the patch accordingly. There have been two main challenges * Opentelemetry spans are designed so that only the process that starts...
Hi @ashb, * I've addressed all your comments, * I created a new migration file, * I added some unit tests for the scheduler changes * I fixed 2 bugs...
@ferruzzi Thanks for the review! I've merged the branch with main. I used to pass the carrier as a parameter to the task command so that it can be accessed...
@ashb @ferruzzi Thank you for all the help! Although the PR is in a good state and the CI is green, I'm going to have to revisit the changes on...
Hello @ferruzzi @ashb, the PR is ready for review. My tests are all passing. I've run them at least 10 times in a row. From what I understand, the new...
Thank you @ashb and @ferruzzi! I couldn't have gotten this through without you.
@potiuk Thanks for pinging me. I have a few patches related to metrics that I'm planning to contribute. This is interesting, I'll add it to my backlog.
Hello everyone, I've started looking into this. I'm doing some testing with both statsd and OTel and so far I've found a lot of metrics that are missing from both....
@paulochf You probably don't see all the metrics with statsd because `2.5.1` is a very old version. I think the earliest supported version at the moment that is also regularly...