apache / apache/parquet-java

Make ParquetFileReader extensible

オープン
#3,006 コメント 11 件 リアクション 0 件 担当者 0 名 GitHub で見る
Type: enhancement
主要言語
Java
スター
3.1k
フォーク
1.6k
平均マージ
3日 12時間
マージ済み PR(30日)
33

説明

### Describe the enhancement requested

I wanted to add a cache for `ParquetFileReader.getColumnIndexStore(int blockIndex)` and I've noticed that I can't access the required `blocks` field from a subclass.
To accomplish this I had to fallback to reflection as can be seen here:
https://github.com/linkedfactory/linkedfactory-pod/blob/c67b155e3fbf76082e2b24fd412efae5838badc6/bundles/io.github.linkedfactory.core/src/main/java/io/github/linkedfactory/core/kvin/parquet/KvinParquet.java#L881

I would propose to improve the extensibility of `ParquetFileReader` by changing certain core members (like `blocks`) to protected or by exposing them through protected getters.

### Component(s)

_No response_

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

調査の方向性

ParquetFileReader、特に getColumnIndexStore(int blockIndex) と blocks フィールドから始め、続いてリンクされている KvinParquet のリフレクション使用箇所を確認して、必要なサブクラスアクセスを特定します。ParquetFileReader のサブクラスがリフレクションなしで要求されたキャッシュを実装でき、選択した protected メンバーまたは getter が文書化され、適切なテストでカバーされていれば完了です。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
java
領域
backend-api-design
issue の種類
機能追加
難易度
4/5
見積もり時間
3〜5日
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
42/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。