Skip to content

Add sequential providers loading to code insights

Warren Gifford requested to merge code-insights-sequential-loading into main

Created by: vovakulikov

Resolve https://github.com/sourcegraph/sourcegraph/issues/19849

Now we have parallel algorithm to load our extension based code insights. This might be not the optimal way to do that because

  • at the start with this approach we have a lot of requests to our backend which is not good because we are bombarding our server with a lot of request.
  • another problem here if we are use HTTP 1.1 which has limitation on parallel requests - 6 or less parallel request allowed but no more in Chrome for example.

You can find table with limitations on different browsers here https://docs.pushtechnology.com/cloud/latest/manual/html/designguide/solution/support/connection_limitations.html

So to avoid this thing we can delay loading of next code insight until our current code insight is resolved. This can help avoid request limitation and decrease time to interactive for first code insights.

Merge request reports