jscheffl commented on PR #38992:
URL: https://github.com/apache/airflow/pull/38992#issuecomment-2120812783
> thanks @jscheffl -- which branch were you on? there are more changes re
mapping on the "mothership" PR (#37851) that are excluded from this PR just for
easier review -- that's the branch you should be on for end-to-end testing. i
cherry pick from that PR branch to make the smaller prs.
>
> i've been doing little bits at a time and this PR is bigger than i'd like,
but it's still not the whole enchalada. this individual PR is still an
incremental addition.
Fully acknowledge. As I was trying to get started into PoC for AIP-69 I
struggled the same and AIP-44 clearly solves all basic problems that I run into
as well. Seems to be this rework is quite complex. Good that you have split up
the "mothership" for incremental review.
As proposed by you I tested on the "mothership" PR #37851 with GIT hash
c1b78c035bf6e8250a12e1a7a0f83f78fcbeaa4e ("run-a-full-task-with-internal-api").
No rebase/merge with main before tests.
So as you cherry-pick into smaller PRs and leaving complexity manageable...
if all is green might still be good to merge the pieces and I assume to make it
working then a few other API calls need to be reworked.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
