divjotarora opened a new pull request, #3749:
URL: https://github.com/apache/parquet-java/pull/3749

   ### Rationale for this change
   
   `RunLengthBitPackingHybridDecoder` allocated integer and byte buffers for 
every packed run. This
   decoder is used for repetition and definition levels and dictionary IDs, so 
repeated packed runs
   created avoidable allocation and GC pressure.
   
   ### What changes are included in this PR?
   
   The decoder now grows and reuses its packed-run buffers, tracks the active 
value position
   independently of buffer capacity, and reads directly from the input stream. 
It preserves existing
   short-input zero padding and EOF behavior. A focused JMH benchmark and 
regression tests cover
   smaller runs following larger runs and truncated final groups.
   
   ### Are these changes tested?
   
   Yes. All 689 `parquet-column` tests pass. The focused benchmark reduced 
normalized allocation from
   13.5–13.75 B/value to 0.002 B/value and improved throughput by 27–37.5% 
across tested bit widths;
   all existing `ReadBenchmarks` workloads also completed successfully.
   
   ### Are there any user-facing changes?
   
   No. This is an internal performance improvement with no API or format 
changes.
   


-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to