Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Collection DML should invoke flush before executing #1395

Closed
beikov opened this issue Dec 1, 2021 · 1 comment · Fixed by #1401
Closed

Collection DML should invoke flush before executing #1395

beikov opened this issue Dec 1, 2021 · 1 comment · Fixed by #1401
Assignees
Labels
component: core kind: bug worth: high Implementing this has a high worth
Milestone

Comments

@beikov
Copy link
Member

beikov commented Dec 1, 2021

It was reported that with a creatable subview contained in an indexed collection, a FK violation has been seen. From the SQL log it is clear that an id value was acquired for the new subview entity while invoking persist, but the insert hasn't been flushed, so the collection DML insert fails because the entry is not there yet.

@beikov beikov added kind: bug component: core worth: high Implementing this has a high worth labels Dec 1, 2021
@beikov beikov added this to the 1.6.4 milestone Dec 1, 2021
@beikov beikov self-assigned this Dec 1, 2021
@beikov beikov modified the milestones: 1.6.4, 1.6.x Dec 8, 2021
@beikov
Copy link
Member Author

beikov commented Dec 8, 2021

It seems we already do the flushing, so I need a reproducer for this from Jason Carreira to further investigate the issue

beikov added a commit to beikov/blaze-persistence that referenced this issue Dec 8, 2021
beikov added a commit to beikov/blaze-persistence that referenced this issue Dec 8, 2021
@beikov beikov modified the milestones: 1.6.x, 1.6.4 Dec 12, 2021
beikov added a commit to beikov/blaze-persistence that referenced this issue Dec 12, 2021
… keys by custom insert/update DML are flushed
beikov added a commit to beikov/blaze-persistence that referenced this issue Dec 12, 2021
… keys by custom insert/update DML are flushed
beikov added a commit to beikov/blaze-persistence that referenced this issue Dec 12, 2021
… keys by custom insert/update DML are flushed
beikov added a commit to beikov/blaze-persistence that referenced this issue Dec 12, 2021
… keys by custom insert/update DML are flushed
beikov added a commit to beikov/blaze-persistence that referenced this issue Dec 12, 2021
… keys by custom insert/update DML are flushed
beikov added a commit to beikov/blaze-persistence that referenced this issue Dec 12, 2021
… keys by custom insert/update DML are flushed
beikov added a commit to beikov/blaze-persistence that referenced this issue Dec 12, 2021
… keys by custom insert/update DML are flushed
beikov added a commit to beikov/blaze-persistence that referenced this issue Dec 12, 2021
… keys by custom insert/update DML are flushed
beikov added a commit that referenced this issue Dec 27, 2021
beikov added a commit that referenced this issue Dec 27, 2021
beikov added a commit to beikov/blaze-persistence that referenced this issue Nov 4, 2024
beikov added a commit to beikov/blaze-persistence that referenced this issue Nov 4, 2024
… keys by custom insert/update DML are flushed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: core kind: bug worth: high Implementing this has a high worth
Projects
None yet
1 participant