Skip to content
Snippets Groups Projects

codeintel: Remove math.Max(X, 1000) from result chunk assignment

Merged Warren Gifford requested to merge ef/18242 into main

Merge request reports

Merged by avatar (Jun 9, 2025 2:32pm UTC)

Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Created by: efritz

    @Strum355 It was originally there since pushing data to SQLite was expensive, but we have better ways to bulk upload into Postgres. I'd rather risk having higher row count in the result chunks table than huge toast payloads because we cap at 1000 (which is super low).

    I don't know how to choose a higher constant limit since we have come enterprise customers with 13GB uploads (no clue how big those result sets get or how many result chunks they should occupy).

Please register or sign in to reply
Loading