- **status**: review --> closed - **Comment**: Merged.
--- ** [tickets:#8214] Compute merge request commits in background** **Status:** closed **Milestone:** unreleased **Labels:** performance **Created:** Fri Jun 29, 2018 08:56 PM UTC by Dave Brondsema **Last Updated:** Mon Jul 02, 2018 06:03 PM UTC **Owner:** Dave Brondsema We added caching to merge requests' list of commits previously (`allura.model.repository.MergeRequest#commits`). But it still can sometimes be slow the very first time. This is particularly a problem when creating the merge request and there's been no chance for it to get cached. * if 'commits' are available (cached), show them on the page just like we do now * if they aren't, return the page without them. * start a background task to get them * use ajax to fetch the bg tasks results when complete, and insert them into the page There is a similar pattern in place already for seeing if the commits are cleanly mergeable (see "merge_task_status") --- Sent from forge-allura.apache.org because dev@allura.apache.org is subscribed to https://forge-allura.apache.org/p/allura/tickets/ To unsubscribe from further messages, a project admin can change settings at https://forge-allura.apache.org/p/allura/admin/tickets/options. Or, if this is a mailing list, you can unsubscribe from the mailing list.
