Patch #44386
closed
Avoid N+1 queries generated by Webhook#setable_projects
Added by Marius BĂLTEANU about 2 months ago.
Updated about 1 month ago.
Description
Extracted from #44337:
Vincent Robert wrote:
The patch also resolves a performance issue by changing one line in Webhook#setable_projects (N+1 queries when filtering lots of projects)
- Status changed from New to Resolved
Committed the performance fix, thanks!
Holger Just wrote in #note-9:
We should also drop the to_a there. Instead, in the before_validation hook we could just pluck the IDs and compare it to the ids of the hook's proposed projects.
Committed this change in r25011.
- Related to Feature #44337: Add an administration page listing all webhooks added
- Status changed from Resolved to Closed
Merged to 7.0-stable branch.
Also available in: Atom
PDF