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

Make use of collection management for updatable entity views in flushing #509

Closed
beikov opened this issue Jan 25, 2018 · 1 comment · Fixed by #659

Comments

@beikov
Copy link
Member

commented Jan 25, 2018

Currently we only use the collection management features for deletion but query flushing would greatly benefit from it as well since we could avoid loading collections altogether.

@beikov beikov added this to the 1.2.0 milestone Jan 25, 2018

@beikov beikov self-assigned this Jan 25, 2018

@beikov beikov modified the milestones: 1.2.0, 1.3.0, 1.6.0 Jan 28, 2018

@beikov

This comment has been minimized.

Copy link
Member Author

commented Apr 11, 2018

Note that we currently flush the entity manager early in ViewCollectionRemoveListener because it is needed to implement orphan removal(#507) properly for collections. Updatable collections currently cause a fallback to entity flushing which will be fixed by the solution for this issue. Don't forget to remove the flushing when implementing this.

beikov added a commit to beikov/blaze-persistence that referenced this issue Sep 24, 2018
beikov added a commit to beikov/blaze-persistence that referenced this issue Sep 25, 2018
beikov added a commit to beikov/blaze-persistence that referenced this issue Oct 11, 2018
beikov added a commit to beikov/blaze-persistence that referenced this issue Oct 14, 2018
beikov added a commit to beikov/blaze-persistence that referenced this issue Oct 16, 2018
beikov added a commit to beikov/blaze-persistence that referenced this issue Oct 17, 2018
beikov added a commit to beikov/blaze-persistence that referenced this issue Oct 17, 2018
beikov added a commit to beikov/blaze-persistence that referenced this issue Oct 17, 2018
beikov added a commit to beikov/blaze-persistence that referenced this issue Oct 17, 2018
beikov added a commit to beikov/blaze-persistence that referenced this issue Oct 17, 2018
beikov added a commit to beikov/blaze-persistence that referenced this issue Oct 18, 2018
beikov added a commit to beikov/blaze-persistence that referenced this issue Oct 18, 2018
beikov added a commit to beikov/blaze-persistence that referenced this issue Oct 18, 2018
beikov added a commit to beikov/blaze-persistence that referenced this issue Oct 19, 2018
beikov added a commit to beikov/blaze-persistence that referenced this issue Oct 19, 2018
beikov added a commit to beikov/blaze-persistence that referenced this issue Oct 19, 2018
beikov added a commit to beikov/blaze-persistence that referenced this issue Oct 19, 2018
beikov added a commit to beikov/blaze-persistence that referenced this issue Oct 19, 2018
beikov added a commit to beikov/blaze-persistence that referenced this issue Oct 20, 2018
beikov added a commit to beikov/blaze-persistence that referenced this issue Oct 20, 2018
beikov added a commit to beikov/blaze-persistence that referenced this issue Oct 20, 2018
beikov added a commit to beikov/blaze-persistence that referenced this issue Oct 21, 2018
beikov added a commit to beikov/blaze-persistence that referenced this issue Oct 21, 2018
beikov added a commit to beikov/blaze-persistence that referenced this issue Oct 21, 2018
beikov added a commit to beikov/blaze-persistence that referenced this issue Oct 21, 2018
beikov added a commit to beikov/blaze-persistence that referenced this issue Oct 21, 2018

@beikov beikov closed this in #659 Nov 8, 2018

beikov added a commit that referenced this issue Nov 8, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant
You can’t perform that action at this time.