Reduce allocation overhead of no-op `GeospatialStatistics.Builder`
メンテナーはふだん 2 日以内に返信
評価
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 初心者へのやさしさ
- 68/100
- issue の種類
- リファクタリング
- 明瞭さ
- おおむね明確
- 活発さ
- 静か
- 技術スタック
- java
- 領域
- performance
調査の方向性
GeospatialStatistics.noopBuilder() から開始し、NoopBuilder、その build() メソッド、および関連するコンストラクターを追跡します。未使用のフィールドがどのように初期化されるかを確認し、merge や abort の動作を変更せずに no-op builder と空の統計結果が再利用されることを検証します。完了条件は、共通パスでこれらのオブジェクトが呼び出しごとに割り当てられなくなることです。
索引モデルが issue の本文から書いたものです。
説明
Describe the enhancement requested
GeospatialStatistics.noopBuilder() is called for every non-geometry column in every row group. Today, each call allocates a new NoopBuilder (plus a WKBReader, BoundingBox, and GeospatialTypes that are never used), and each build() allocates a new GeospatialStatistics(null, null) that is immediately discarded.
Since NoopBuilder carries no state and all its methods are no-ops, it can be a singleton. Its build() result is equally stateless (null fields, with merge/abort guarded by null checks), so it can be cached too.
Proposed Changes
- Make
NoopBuildera singleton via a staticINSTANCEfield - Cache
build()output as a staticEMPTYinstance - Add a package-private super constructor that skips field initialization so the singleton doesn't allocate unused objects
This eliminates per-column-per-row-group allocations on the common path where geospatial statistics are disabled.
Component(s)
Core
- 主要言語
- Java
- スター
- 3.1k
- フォーク
- 1.6k
- 平均マージ
- 4日 5時間
- マージ済み PR（30日）
- 30
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドなし
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
apache/parquet-java のほかの issue
-
Row-group copying collides for distinct column paths with the same dot string対応中かも @costas-db が 7 日前に担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100
apache/parquet-java#3829 ·
メンテナーはふだん 2 日以内に返信
-
Binary statistics truncation test ignores its configured truncation length対応中かも @dhruv-15-03 が 9 日前に担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
apache/parquet-java#3820 ·
メンテナーはふだん 2 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
apache/parquet-java#3767 ·
メンテナーはふだん 2 日以内に返信
-
Remove duplicate LICENSE and NOTICE files from benchmark JARs対応中かも @efegokdemir が 12 日前に担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
apache/parquet-java#3695 · コメント 1 件 ·
メンテナーはふだん 2 日以内に返信
-
Close input readers when ParquetRewriter setup fails対応中かも @anxkhn が 83 日前に担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
apache/parquet-java#3667 ·
メンテナーはふだん 2 日以内に返信
apache/parquet-java の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
-
area/docs
難易度 1/5 1時間未満 初心者へのやさしさ 88/100
メンテナーはふだん 1 日以内に返信
-
BoxChart rejects valid List.of data with NullPointerException対応中かも @PHJ2000 が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100