[PARQUET-1364] Column Indexes: Invalid row indexes for pages starting with nulls

• Type: Sub-task
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Labels: pull-request-available
• Assignee: Gabor Szadovszky
• Reporter: Gabor Szadovszky
• Created: 31/Jul/18 13:37
• Updated: 23/Jun/24 03:30
• Resolved: 03/Aug/18 05:39

Description

The current implementation for writing managing row indexes for the pages is not reliable. There is a logic MessageColumnIO which caches null values and flush them just before opening a new group. This logic might cause starting pages with these cached nulls which are not correctly counted in the written rows so the rowIndexes are incorrect. It does not cause any issues if all the pages are read continuously put it is a huge problem for column index based filtering.
The implementation described above is really complicated and would not like to redesign because of the mentioned issue. It is easier to simply count the 0 repetition levels as record boundaries at the column writer level.

Issue Links

• blocks: PARQUET-1310 Column indexes: Filtering
• links to: GitHub Pull Request #507