[jira] [Comment Edited] (ARROW-17904) [C++] Parquet support read page with crc32 checking

["Xuwei Fu (Jira)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/ARROW-17904?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17614587#comment-17614587 ] 

Xuwei Fu edited comment on ARROW-17904 at 10/9/22 8:01 AM:
-----------------------------------------------------------

It's ok, I'll try to use zlib. Should I extract the zlib's `crc32` like mysql, or keeping `ARROW_WITH_ZLIB ON` when in arrow and parquet? [~kou] [~apitrou] 


was (Author: JIRAUSER294084):
It's ok, I'll try to use zlib. Should I extract the zlib's `crc32` like mysql, or keeping `ARROW_WITH_ZLIB ON` when in arrow and parquet? 

> [C++] Parquet support read page with crc32 checking
> ---------------------------------------------------
>
>                 Key: ARROW-17904
>                 URL: https://issues.apache.org/jira/browse/ARROW-17904
>             Project: Apache Arrow
>          Issue Type: Improvement
>          Components: Parquet
>    Affects Versions: 9.0.0
>            Reporter: Xuwei Fu
>            Assignee: Xuwei Fu
>            Priority: Major
>              Labels: pull-request-available
>             Fix For: 11.0.0
>
>          Time Spent: 0.5h
>  Remaining Estimate: 0h
>
> Currently, C++'s Parquet support write page with checksum, but `ReadPage` doesn't have check any checksum. And I would like to fix it
> I'd like to split this patch to different parts:
>  # Implement the crc in DataPageV1, which requires a write crc config, counting crc in read mode, crc verification, migrate testing from parquet mr
>  # Implement crc in DataPageV2 and Dict



--
This message was sent by Atlassian Jira
(v8.20.10#820010)