Setting parquet.thrift.string.size.limit = -1 incorrectly throws
- 主要言語
- Java
- スター
- 3.1k
- フォーク
- 1.6k
- 平均マージ
- 3日 12時間
- マージ済み PR(30日)
- 33
説明
### Describe the bug, including details regarding any error messages, version, and platform.
https://github.com/apache/parquet-java/pull/3359 added a `parquet.thrift.string.size.limit` configuration to allow specifying a custom max Thrift message size. If unspecified or explicitly set to -1, the default (100 MB) is used. The config validation code introduced a minor bug [here](https://github.com/apache/parquet-java/blob/8931c1c55f1fba399dd75139f75bcde0b84137c0/parquet-format-structures/src/main/java/org/apache/parquet/format/Util.java#L420): if the value is explicitly set to -1, `maxMessageSize` is correctly set to the default, but the `configuredMaxMessageSize <= 0` branch will still throw an exception.
### Component(s)
_No response_
コントリビューションガイド
このリポジトリのコントリビューションガイドは索引されていません
調査の方向性
parquet-format-structures/src/main/java/org/apache/parquet/format/Util.java の、420 行目付近にあるリンク先の検証コードから始めます。parquet.thrift.string.size.limit が明示的に -1 に設定されているケースを再現し、例外をスローせずに 100 MB のデフォルト値が使用されることを確認します。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- java
- 領域
- backend
- issue の種類
- バグ
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 活発さ
- 静か
- 明瞭さ
- 明確に書かれている
- 初心者へのやさしさ
- 76/100