Jed Cunningham
Jed Cunningham
Let's see. So #28871 attempted to fix the the missing "TaskInstance Finished" log line, and #30872 was a fix of that fix. With that if block from #30872 gone, I'd...
I lean toward 1. More explicit. I'm not overly concerned with the name problem, so your suggestion of dag_id-task_id-random works fine. We can put all the identifying info in labels...
Thanks for testing the RC @bdsoha @humit0 @LucasRoesler @GrumpyCat51 @danielcarletti @Miretpl and @potiuk. The release is underway!
My 2c, giving bundles somewhere to store this extra stuff makes sense. I'd rather we let the bundles own id creation though - if there is one that is meaningful...
> IMHO the interface should expect a string payload back from the bundle provider which is the representation of the dag bundle version (which can be whatever mechanism the provider...
I don't see an issue with batching them. @potiuk I'm slightly hesitant to set the minimum purely based on "~6 months old" though. Are we planning to moving those lower...
I haven't looked at either PR, but #56589 has been open for a month with this same feature.