perf(persistence): Move thread messages pagination to SQL#2750
perf(persistence): Move thread messages pagination to SQL#2750VelikovPetar wants to merge 1 commit 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 65.96% 65.98% +0.02%
==========================================
Files 425 425
Lines 26926 26946 +20
==========================================
+ Hits 17761 17781 +20
Misses 9165 9165 ☔ 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.