grafana / pyroscope

Continuous Profiling Platform. Debug performance issues down to a single line of code
https://grafana.com/oss/pyroscope/
GNU Affero General Public License v3.0
10.13k stars 614 forks source link

feat: add GetCommits endpoint for efficient multiple commits fetching #3529

Closed marcsanmi closed 2 months ago

marcsanmi commented 2 months ago

This PR introduces a significant enhancement to the VCS service by adding a new endpoint for parallel fetching of multiple commits. This change improves performance and efficiency when retrieving commit information for multiple refs while maintaining full backwards compatibility

Key changes

Main benefits

  1. Reduced network overhead: The frontend can now request multiple commits in a single API call, significantly reducing the number of HTTP requests.
  2. Improved performance: Parallel fetching of commits leads to faster overall response times, especially when dealing with multiple refs.
  3. Better error handling: The service now provides detailed information about both successful and failed commit fetches.
  4. Backwards compatibility: Existing integrations using the GetCommit endpoint remain fully functional and unchanged.

Frontend impact