From b388b81f7b8f046317cb838514b04227a26dd8e1 Mon Sep 17 00:00:00 2001 From: mvdbeek Date: Mon, 13 May 2024 15:11:57 +0200 Subject: [PATCH] Drop unnecessary commit() call --- lib/galaxy/job_execution/actions/post.py | 3 --- 1 file changed, 3 deletions(-) diff --git a/lib/galaxy/job_execution/actions/post.py b/lib/galaxy/job_execution/actions/post.py index 10c1e3a57071..8fd85e03dd47 100644 --- a/lib/galaxy/job_execution/actions/post.py +++ b/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