Skip to content

Conversation

brendan-kellam
Copy link
Contributor

This PR optimizes the getRepos action by removing a JOIN we were doing on the Connection table. In my brief testing, this improves query performance by 2-3x, which should improve initial page load times for large instances.

The side-effect of this is that certain UX elements (repository table, progress/error tracker) no longer link back to a connection.

Copy link

coderabbitai bot commented Sep 20, 2025

Important

Review skipped

Auto reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch bkellam/remove_join_on_connections

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

This comment has been minimized.

@brendan-kellam brendan-kellam merged commit a76ae68 into main Sep 20, 2025
9 checks passed
@brendan-kellam brendan-kellam deleted the bkellam/remove_join_on_connections branch September 20, 2025 18:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants