jerolba opened a new issue, #3226:
URL: https://github.com/apache/parquet-java/issues/3226

   ### Describe the enhancement requested
   
   Profiling the load of a Parquet file with Java Mission Control, I've noticed 
that `InternalParquetRecordReader`  
[LongStream](https://github.com/apache/parquet-java/blob/1f1e07bbf750fba228851c2d63470c3da5726831/parquet-hadoop/src/main/java/org/apache/parquet/hadoop/InternalParquetRecordReader.java#L323)
 consumes relevant amount of time.
   
   This `LongStream` can be replaced with a simpler Long Iterator that iterates 
from 0 to `pages.getRowCount()`.
   
   To measure the overhead I've created a test project that overwrites 
`InternalParquetRecordReader` implementation with a Long Iterator: 
https://github.com/jerolba/parquet-rowindexiterator 
   
   The execution time is sensitive to the context of the JVM, but running the 
benchmark multiple times shows that LongStream is slower than LongIterator, 
between 1% and 4% depending on the run.
   
   ### Component(s)
   
   _No response_


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