junyeong0619 commented on PR #66022: URL: https://github.com/apache/airflow/pull/66022#issuecomment-4416153622
Thanks for the thorough review, @potiuk! Before I dive into the inline comments, I'd like to confirm whether this PR can move forward, since your last comment raises a more fundamental concern about the design itself. The scoping question (metastore-only vs. secrets backend) was discussed on the dev mailing list: https://lists.apache.org/thread/m9on5f334rg7ykk33vgpvx9znvoxcvm1 To summarize the outcome: - @amoghdesai explicitly agreed with the metastore-only scope and suggested the lazy-proxy design that the current PR implements. - @jscheffl and @ashb raised concerns (confusion risk with secrets backends, AIP-103 overlap), which I addressed in-thread. No blocking objections were raised afterward. That said, if there are additional concerns beyond what was discussed on the ML — or if @kaxil sees this differently — I'd rather pause and align before iterating on the inline comments. --- Drafted-by: Claude Code (Opus 4.7); reviewed by @junyeong0619 before posting -- 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]
