Skip to content

Commit

Permalink
fix: Use AsyncioRequestsTransport as default async transport
Browse files Browse the repository at this point in the history
    Removing `aiohttp_retry` means we don't have a guarantee `aiohttp`
  • Loading branch information
kdestin committed Aug 22, 2024
1 parent a49331f commit e077ca4
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/promptflow-evals/promptflow/evals/_http_utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -21,8 +21,8 @@
UserAgentPolicy,
)
from azure.core.pipeline.transport import ( # pylint: disable=non-abstract-transport-import,no-name-in-module
AioHttpTransport,
AsyncHttpTransport,
AsyncioRequestsTransport,
HttpTransport,
RequestsTransport,
)
Expand Down Expand Up @@ -294,7 +294,7 @@ def __init__(
# AioHttpTransport normally should not be imported outside of azure.core, since transports
# are meant to be user configurable.
# AioHttpTransport is only used in this file as the default transport when not user specified.
transport=transport or AioHttpTransport(**kwargs),
transport=transport or AsyncioRequestsTransport(**kwargs),
policies=[
config.headers_policy,
config.user_agent_policy,
Expand Down

0 comments on commit e077ca4

Please sign in to comment.