Skip to content

Refactor: Remove redundant dict.keys() call in airflow/api_connexion/endpoints/pool_endpoint.py#33158

Merged
potiuk merged 1 commit into
apache:mainfrom
eumiro:keys-api-connexion
Aug 7, 2023
Merged

Refactor: Remove redundant dict.keys() call in airflow/api_connexion/endpoints/pool_endpoint.py#33158
potiuk merged 1 commit into
apache:mainfrom
eumiro:keys-api-connexion

Conversation

@eumiro

@eumiro eumiro commented Aug 6, 2023

Copy link
Copy Markdown
Contributor

No description provided.

@boring-cyborg boring-cyborg Bot added the area:API Airflow's REST/HTTP API label Aug 6, 2023
@eumiro
eumiro force-pushed the keys-api-connexion branch from 4cdd70f to d65eccc Compare August 6, 2023 18:37

@uranusjr uranusjr left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The set cast can be avoided entirely.

Comment thread airflow/api_connexion/endpoints/pool_endpoint.py Outdated
Comment thread airflow/api_connexion/endpoints/pool_endpoint.py Outdated
@eumiro
eumiro force-pushed the keys-api-connexion branch from d65eccc to e52b736 Compare August 7, 2023 17:02
@eumiro
eumiro requested a review from uranusjr August 7, 2023 17:03
@eumiro
eumiro force-pushed the keys-api-connexion branch from e52b736 to c407b03 Compare August 7, 2023 17:04
@potiuk
potiuk merged commit ba9d4e4 into apache:main Aug 7, 2023
@eumiro
eumiro deleted the keys-api-connexion branch August 7, 2023 18:54
@ephraimbuddy ephraimbuddy added the type:misc/internal Changelog: Misc changes that should appear in change log label Aug 8, 2023
@ephraimbuddy ephraimbuddy added this to the Airflow 2.7.0 milestone Aug 8, 2023
ephraimbuddy pushed a commit that referenced this pull request Aug 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:API Airflow's REST/HTTP API type:misc/internal Changelog: Misc changes that should appear in change log

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants