GitHub user emkornfield added a comment to the discussion: Dictionary page offset logic
> I think the current code sets the col_start to the data_page_offset since we > have the condition column_metadata->dictionary_page_offset() > 0 ... so I'm > not sure how line 189 returns an error for a negative dictionary offset. You are right, I read it too quickly, I think the answer is java wrote out zero values. > If the data_page_offset is 0 (no data page), it will prevent us from setting > col_start to the dictionary page offset (for valid positive dictionary > offset). This will trigger an error. I'm not sure what writers end up populating this field with if the values are no data pages, but ideally we wouldn't get here because we already no there are zero rows from the row group metadata? GitHub link: https://github.com/apache/arrow/discussions/48184#discussioncomment-15018958 ---- This is an automatically sent email for [email protected]. To unsubscribe, please send an email to: [email protected]
