dabla commented on PR #35591: URL: https://github.com/apache/airflow/pull/35591#issuecomment-2112380773
> > It’s a bit hard to split as all modifications are linked to each other. I was already happy I was able to pull it off imho ☺️ > > I know, but maybe worth trying. There is a lot to review here and someone will have to find quite a bit of chunk of free time to review it, I am going to Pycon next week and for sure will have no time to review this one and since it is a crucial HTTP hook it needs more than one person and quality review, so I dismissed my past review and hopefully there will be someone who can review it promptly enough I will already create a new PR for the change I did concerning the extra_dejson in the Connection. Ofc that addition in the Connection class won't be used in Airflow yet (apart from the test), but then it would already be a separate PR which could then make this one a bit simplier. -- 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]
