D. Ferruzzi

Results 57 comments of D. Ferruzzi

I'm really not sure what that failing static check is trying to tell me. Wouldn't mind a hand on that if someone knows what it means.

> Adding a how-to-guide section in provider.yaml which references the new doc should solve it I SEE! Some of the integrations list a how-to and others don't, I'll add that...

Alright. @vincbeck I believe I have address the concerns you raised I just ran the system test end to end three times: ``` default_args={deferrable=True, wait_for_completion=False} ``` ``` default_args={deferrable=False, wait_for_completion=True} ```...

Alright, got all the tests passing and rebased since it's been a few days. That was an adventure, sorry about that.

how could the change in [6d5cdd2](https://github.com/apache/airflow/pull/39245/commits/6d5cdd27fecea43655826eb3212d022befc6c581) have broken three test which were all passing before that commit? It's copy and pasted dag declaration from the rest of our system tests.

Assuming the CI goes green, I'm done. @vincbeck - you may want to have a peek at the changes since you approved, or not. @Taragolis - I ended up doing...

@wlinamchurch You still need one more approval, perhaps drop a message in the slack #contributors channel asking for another set of eyes.

Congrats! Thanks for your contribution and your patience!

Not sure it really qualifies as a bug, but it didn't really feel like a "feature request" either. Feel free to adjust my tagging. I've added "good first issue" because...

I think the easiest answer would be to convert the existing metrics tags dicts to ordereddicts, then in the statsd_logger module you can join them and know they'll always be...