apache / apache/arrow-rs

[Parquet] Add metadata heap size test that tracks allocations

Open
#8,924 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Rust
Stars
3.6k
Forks
1.3k
Avg merge
2d 14h
Merged PRs (30d)
167

Description

See relevant comments here: https://github.com/apache/arrow-rs/pull/8898#issuecomment-3562631348

There are tests that use hard-coded expected memory sizes (eg [here](https://github.com/apache/arrow-rs/blob/1c90efe717c1a2f99a0a97d6e9c93cd913eaca3c/parquet/src/file/metadata/mod.rs#L1815)), but these expected sizes could become incorrect if the metadata layout changes.

Contributor guide

Open the contributing guide

Research direction

Start in parquet/src/file/metadata/mod.rs around the hard-coded metadata memory-size test at the linked location, then read the referenced comments on pull request 8898. Add a metadata heap-size test that tracks allocations and remains valid when the metadata layout changes; existing metadata tests should continue to pass.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
data-engineering
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
48/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.