kalluripradeep commented on code in PR #59938:
URL: https://github.com/apache/airflow/pull/59938#discussion_r2655581544
##########
airflow-core/src/airflow/models/pool.py:
##########
@@ -389,4 +399,4 @@ def get_name_to_team_name_mapping(
pool_names: list[str], session: Session = NEW_SESSION
) -> dict[str, str | None]:
stmt = select(Pool.pool,
Pool.team_name).where(Pool.pool.in_(pool_names))
- return {pool: team_name for pool, team_name in session.execute(stmt)}
+ return {pool: team_name for pool, team_name in session.execute(stmt)}
Review Comment:
@potiuk Fixed both issues:
1. Removed the accidental fix_quote.py file
2. Fixed the duplicate return statement (it was from an earlier commit)
Also fixed the LoggingMixin usage by switching to a module-level logger as
per the pre-commit checks.
Thanks for the quick review!
--
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]