dabla commented on code in PR #71350:
URL: https://github.com/apache/airflow/pull/71350#discussion_r3835873695
##########
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:
It's a choice, for MSGraph it was an obvious choice as the Python SDK was
async only, so it made sense. Here we can choose, hence why I brought it up.
Still I think polling should always be done in deferred mode and thus via a
trigger to avoid blocking the worker. As this is a new hook/operator, I think
it's an opportunity to make that deliberate choice as well.
--
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]
