aaron-y-chen commented on code in PR #71350:
URL: https://github.com/apache/airflow/pull/71350#discussion_r3835517492
##########
providers/microsoft/azure/pyproject.toml:
##########
@@ -99,6 +99,7 @@ dependencies = [
# It was added in https://github.com/apache/airflow/pull/47990/files
# maybe this should be set from upstream
"msal-extensions>=1.3.0",
+ "requests>=2.32.0,<3",
Review Comment:
Refactor done, and passed the e2e test:
<img width="1431" height="547" alt="image"
src="https://github.com/user-attachments/assets/b886d6ac-b0bf-429a-9e93-533451b3eecb";
/>
<img width="1908" height="583" alt="image"
src="https://github.com/user-attachments/assets/7e8ceaa0-73ab-41c4-b3be-c3b82d3b189d";
/>
I kept `asyncio.to_thread()` because `get_refresh_status()` and
`ClientSecretCredential.get_token()` are synchronous and are also used by the
Operator, Sensor, and Hook polling paths. Offloading the call keeps the
triggerer event loop non-blocking without duplicating the Hook API, and follows
the existing pattern in `triggers/batch.py`.
--
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]
