Merge pull request #18131 from mvdbeek/no_commit_in_delete_pja

[24.0] Don't commit in ``DeleteIntermediatesAction``

lib/galaxy/job_execution/actions/post.py

@@ -8,7 +8,6 @@
 from markupsafe import escape
 
 from galaxy.model import PostJobActionAssociation
-from galaxy.model.base import transaction
 from galaxy.util import send_mail
 from galaxy.util.custom_logging import get_logger
 
@@ -370,8 +369,6 @@ def execute(cls, app, sa_session, action, job, replacement_dict, final_job_state
         # POTENTIAL ISSUES:  When many outputs are being finish()ed
         # concurrently, sometimes non-terminal steps won't be cleaned up
         # because of the lag in job state updates.
-        with transaction(sa_session):
-            sa_session.commit()
         if not job.workflow_invocation_step:
             log.debug("This job is not part of a workflow invocation, delete intermediates aborted.")
             return