perf(persistence): Move thread messages pagination to SQL#2750
perf(persistence): Move thread messages pagination to SQL#2750VelikovPetar wants to merge 2 commits into
Conversation
Back-port of #2688 to v9. Refactors `MessageDao.getThreadMessagesByParentId` to filter and paginate thread replies at the SQL level instead of loading the full thread into memory, and adds support for all four `PaginationParams` cursor variants (`lessThan` / `lessThanOrEqual` / `greaterThan` / `greaterThanOrEqual`) with a `(createdAt, id)` tiebreaker. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## v9 #2750 +/- ##
=====================================
Coverage ? 66.00%
=====================================
Files ? 425
Lines ? 26970
Branches ? 0
=====================================
Hits ? 17802
Misses ? 9168
Partials ? 0 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Submit a pull request
Linear: FLU-487
Github Issue: #
CLA
Description of the pull request
Back-port of #2688 to v9. Refactors
MessageDao.getThreadMessagesByParentIdto filter and paginate thread replies at the SQL level instead of loading the full thread into memory, and adds support for all fourPaginationParamscursor variants (lessThan/lessThanOrEqual/greaterThan/greaterThanOrEqual) with a(createdAt, id)tiebreaker.