potiuk commented on PR #66022: URL: https://github.com/apache/airflow/pull/66022#issuecomment-4416993778
> 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. Thanks for bringing the thread.. No concerns after you quoted 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]
