Optimize Element.remove_child
Requires !1406 (merged)
While spending a lot of time updating all the SQL assertions, I noticed Element.remove_child
made a lot of unnecessary queries and loaded unnecessary columns and models. I made some quick changes that should avoid loading child element UUIDs in RAM and avoid making one SQL query for each child element. There is definitely room for more improvement should we ever switch to full-SQL path manipulation methods, but this is a quick win.
Edited by Erwan Rouchet
Merge request reports
Activity
requested review from @babadie
added 5 commits
-
8524005f...8f2650f2 - 4 commits from branch
master
- a7be6209 - Optimize Element.remove_child
-
8524005f...8f2650f2 - 4 commits from branch
mentioned in commit 32d7af3f
Please register or sign in to reply