abhishekmauryaKsolves commented on PR #70647: URL: https://github.com/apache/airflow/pull/70647#issuecomment-5139324632
Reverted batch_is_authorized_connection, batch_is_authorized_dag, batch_is_authorized_pool, batch_is_authorized_variable, filter_authorized_connections, filter_authorized_pools, and filter_authorized_variables back to the original N-parallel-calls approach (ThreadPoolExecutor + single_flight caching) — fine-grained per-resource authorization is fully restored, no behavior change from before this PR touched them. _is_batch_authorized stays unused by this PR now (it's only exercised by the pre-existing filter_authorized_menu_items, which is safe to batch since menu items are individually registered Keycloak resources). Net result: this PR now only adds the new batch_is_authorized_* / filter_authorized_* public API surface (per #70582), with the same call pattern and semantics as the existing is_authorized_* methods underneath. All 248 tests pass. -- 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]
