Jackie-Jiang opened a new pull request #5816:
URL: https://github.com/apache/incubator-pinot/pull/5816


   ## Description
   Currently for var-byte raw index, we always pre-allocate the chunk buffer 
even for uncompressed data (the buffer could be huge if the index contains 
large size value). When reading values, we first copy the data into the chunk 
buffer, then read from the buffer. This could cause unnecessary overhead on 
copying the data as well as allocating the direct memory, and can even cause 
OOM if the buffer size is too big. The chunk buffer is needed for compressed 
data in order to decompress it, but not necessary for uncompressed data as we 
can directly read from the data buffer.
   
   This PR enhances the VarByteChunkSVForwardIndexReader to directly read from 
the data buffer for uncompressed data, and avoid the overhead of the chunk 
buffer.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org

Reply via email to