[TRI-3867] Updating lots of environment variables at once fails with a transaction timeout

When we bulk upsert environment variables we wrap the entire thing in a database transaction. If there are a lot of env vars (e.g. more than 100) then the transaction can timeout.

You can see in the current code that the transaction is wrapped around the for loop:

trigger.dev/apps/webapp/app/v3/environmentVariables/environmentVariablesRepository.server.ts

Line 138 in feb4fcd

const result = await $transaction(this.prismaClient, async (tx) => {

Instead we should move that transaction inside the for loop. We still need the transaction so we always create all the resources for each env var and don't leave any of them dangling.

_TRI-3867

@matt-aitken

hey @matt-aitken working in this issue

@matt-aitken

@matt-aitken Checkout this #1413

Navigation Menu

Repository navigation

[TRI-3867] Updating lots of environment variables at once fails with a transaction timeout #1411

58 remaining items

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

[TRI-3867] Updating lots of environment variables at once fails with a transaction timeout #1411

Description

Activity

yadavshubham01 commented on Oct 16, 2024

yadavshubham01 commented on Oct 16, 2024

58 remaining items

Metadata

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Issue actions