[PR #2210] [MERGED] fix: improve sealing speed #2124

Closed
opened 2026-02-26 20:32:32 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/documenso/documenso/pull/2210
Author: @dguyen
Created: 11/19/2025
Status: Merged
Merged: 11/19/2025
Merged by: @dguyen

Base: mainHead: fix/improve-sealing-speed


📝 Commits (1)

  • 31b6d41 fix: improve sealing speed

📊 Changes

1 file changed (+103 additions, -164 deletions)

View changed files

📝 packages/lib/jobs/definitions/internal/seal-document.handler.ts (+103 -164)

📄 Description

Description

Fixes issues with long running sealing jobs

Main findings:

  • Each step costs time due to the scheduler taking time to pick them up
  • Unnecessary async code reruns between steps

Changes made:

  • Reduce step count down to 2

Local testing:

  • Using Inngest cloud, sealing jobs duration cut in half
  • Majority of time was downloading/uploading files to S3, so could be faster on production

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/documenso/documenso/pull/2210 **Author:** [@dguyen](https://github.com/dguyen) **Created:** 11/19/2025 **Status:** ✅ Merged **Merged:** 11/19/2025 **Merged by:** [@dguyen](https://github.com/dguyen) **Base:** `main` ← **Head:** `fix/improve-sealing-speed` --- ### 📝 Commits (1) - [`31b6d41`](https://github.com/documenso/documenso/commit/31b6d41cd2cc330896d36e0780dc5199fe734b21) fix: improve sealing speed ### 📊 Changes **1 file changed** (+103 additions, -164 deletions) <details> <summary>View changed files</summary> 📝 `packages/lib/jobs/definitions/internal/seal-document.handler.ts` (+103 -164) </details> ### 📄 Description ## Description Fixes issues with long running sealing jobs Main findings: - Each step costs time due to the scheduler taking time to pick them up - Unnecessary async code reruns between steps Changes made: - Reduce step count down to 2 Local testing: - Using Inngest cloud, sealing jobs duration cut in half - Majority of time was downloading/uploading files to S3, so could be faster on production --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-26 20:32:32 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/documenso#2124
No description provided.