jscheffl commented on code in PR #57452:
URL: https://github.com/apache/airflow/pull/57452#discussion_r2471071952
##########
airflow-core/src/airflow/api_fastapi/core_api/routes/public/backfills.py:
##########
@@ -162,6 +162,7 @@ def unpause_backfill(backfill_id: NonNegativeInt, session:
SessionDep) -> Backfi
raise HTTPException(status.HTTP_409_CONFLICT, "Backfill is already
completed.")
if b.is_paused:
b.is_paused = False
+ session.commit()
Review Comment:
As long as no exception is raised the DB transaction is committing per
default. As long as there is no "bug" reported we do not need an explicit
commit() in my view.
```suggestion
```
--
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]
