Performance concern: fillHoles() method and read buffer expansion efficiency.

Hi everyone,

I have the following use case: I’m benchmarking the read throughput performance when dealing with a large number of non-dictionary string columns (300 columns). Based on the profiler output (see the attached picture), I’ve noticed that a significant amount of time is spent in the fillHoles() method, which is part of the read buffer expansion process.

My question is: why is the buffer filled one element at a time instead of using a bulk operation? Wouldn’t a batch approach be more efficient?

Looking forward to your insights. Thanks!


read buffer の拡張パスにある fillHoles() から始め、issue で説明されている profiler image と 300-column benchmark のコンテキストを確認してください。現在の要素ごとの filling と、同じ workload を使った bulk approach を比較してください。完了の条件は、パフォーマンス上の懸念について測定に基づく結論が得られ、比較結果がそれを支持する場合に限って焦点を絞った変更を行うことです。

索引モデルが issue の本文から書いたものです。