apache / apache/parquet-java

Setting parquet.thrift.string.size.limit = -1 incorrectly throws

Offen Anfängerfreundlich
#3,587 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Type: bug
Vorherrschende Sprache
Java
Sterne
3.1k
Forks
1.6k
Ø Merge
3 T. 12 Std.
Gemergte PRs (30 T.)
33

Beschreibung

### 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_

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Rechercherichtung

Beginne in parquet-format-structures/src/main/java/org/apache/parquet/format/Util.java bei der verknüpften Validierungslogik etwa in Zeile 420. Stelle den Fall nach, in dem parquet.thrift.string.size.limit explizit auf -1 gesetzt ist, und überprüfe anschließend, dass der 100-MB-Standardwert verwendet wird, ohne eine Exception auszulösen.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
java
Bereich
backend
Issue-Typ
Bug
Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Aktivitätsstatus
Ruhig
Klarheit
Klar beschrieben
Anfängerfreundlichkeit
76/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.