pinot icon indicating copy to clipboard operation
pinot copied to clipboard

Improve splitPart function

Open gortiz opened this issue 3 months ago • 2 comments

The split part function (link) is allocation-heavy.

It first calculates all splits and then only returns the one indexed as requested by its argument. That is very allocation expensive when the index is very small and the separator is repeated several times.

gortiz avatar Dec 12 '25 15:12 gortiz

@gortiz Can you please assign this to me ?

vinodbaba avatar Dec 13 '25 14:12 vinodbaba

Sure!

gortiz avatar Dec 15 '25 07:12 gortiz

Hi @gortiz could you assign this to me. I have raised a PR for this performance fix : https://github.com/apache/pinot/pull/17428

pie-humble-pie avatar Dec 25 '25 12:12 pie-humble-pie