codeintel: Remove math.Max(X, 1000) from result chunk assignment
Created by: efritz
Fixes https://github.com/sourcegraph/sourcegraph/issues/18242.
Merge request reports
Activity
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