Daniel Standish

Results 109 comments of Daniel Standish

@uranusjr do you have any concerns with this PR?

@uranusjr you want to have a look see here?

> > It doesn't (either completely or at all) include the dag object or its groups. > > And… we somehow don’t want it to do that? I feel we...

but like ... also... @uranusjr, using this example... there actually are some reasonable arguments for not serializing the complete task object. for one it's more efficient. why serialize the full...

closing in favor of https://github.com/apache/airflow/pull/39604

ok @potiuk @vincbeck @uranusjr -- i think this is ready for another look. This "base" PR is the first 4 commits from the mothership PR (https://github.com/apache/airflow/pull/37851) After this main PR...

> If it helps let me know then I could make a full test as I am failing exactly on this point maight be easy. Otherwise I assume some more...

Thanks yeah you mean like help with practical testing like trying things out. I’ve done a lot of it so I feel ok about it so not necessary unless you...

thanks @jscheffl -- which branch were you on? there are more changes re mapping on the "mothership" PR (https://github.com/apache/airflow/pull/37851) that are excluded from this PR just for easier review --...

> missing in internal API or will Subdag just not be supported? etc ah yeah those scenarios are good to add to my testing dag. maybe you could share your...