potiuk commented on PR #38180: URL: https://github.com/apache/airflow/pull/38180#issuecomment-2021452882
Yes. It's very bad to keep you in a limbo like that especially after all the attempt you've made @kzosabe . Apologies for that in the name of all other maintainers. You seem to have a valid annoying issue that we generally avoided to handle. I have no idea if it is right or wrong. This all seems from the implementation in #30612 and it seem to be somewhat broken anyway as indicated in https://github.com/apache/airflow/pull/30612#issuecomment-1980985307 and PR to likely fix some of that in #37936. Maybe @vandonr as the original author can chime in an comment on those. Seem that none of the other maintainers involved in the past discussions have picked up the discussion, but It's lkely better to avoid harm even if the discussion is not finished (cc: @hussein-awala @uranusjr). I will ping them to see if they can be fixed in the original PRs, but for now I see no harm at all to change that warning into a debug mode. Happy to accept the change for 2.9.0, but: a) please change it to debug. b) please leave a comment explaining that this is temporary measure until the PR/issue is fixed. Please ping me when it's done and I will merge it. -- 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]
