Skip to content

Conversation

@jussisaurio
Copy link
Collaborator

Description

Closes #4146

Motivation and context

panics are bad

AI Disclosure

none used

Copy link

@turso-bot turso-bot bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please review @pedrocarlo

Copy link
Contributor

@pedrocarlo pedrocarlo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't see a problem with it. I imagine the tests failures are due to something else

@Pavan-Nambi
Copy link
Contributor

Pavan-Nambi commented Dec 12, 2025

with thiis fix i'm getting this error after hang/waiting.. for few mins, ig we r slow., but i would assume this is a sim bug not this pr, i would take a look at thsi evening when i get back.

also note that we might have to consider adding --experimental to triggers.. before 0.4 #4022, i still have to rebase with #4145, it touches lot of code i used..

before fix:
sim-2

after this fix:

image

@Pavan-Nambi
Copy link
Contributor

but i would assume this is a sim bug not this pr, i would take a look at thsi evening when i get back.

update: it is a sim bug not db.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

index update during before update trigger causes Bitfield<4> out-of-bounds panic

5 participants