apache / apache/arrow-java

[Java] Dummy NPEs in DefaultAllocationManagerOption

Offen
#258 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Type: enhancement
Vorherrschende Sprache
Java
Sterne
94
Forks
152
Ø Merge
3 T. 16 Std.
Gemergte PRs (30 T.)
11

Beschreibung

Looking at DefaultAllocationManagerOption [1], I see that we could prevent dummy NPEs when trying to convert an often not-set environment option into enums.
In my case, it was a bit painful to have this error, even if properly caught, because it was polluting debug when looking for unexpected NPEs. This was raising false positives.
 
I was wondering if you were open to a pull request to change this little inconvenience?
 
[1]

**Reporter**: [Olivier Peyrusse](https://issues.apache.org/jira/browse/ARROW-13697)

**Note**: *This issue was originally created as [ARROW-13697](https://issues.apache.org/jira/browse/ARROW-13697). Please see the [migration documentation](https://github.com/apache/arrow/issues/14542) for further details.*

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Beginne damit, java/memory/memory-core/src/main/java/org/apache/arrow/memory/DefaultAllocationManagerOption.java ungefähr in den Zeilen 64-82 zu lesen, wobei du dich auf die Konvertierung nicht gesetzter Umgebungsoptionen in Enums konzentrierst. Bestätige das aktuelle Verhalten für eine nicht gesetzte Option und überprüfe, dass die daraus resultierende Behandlung keine Dummy-NullPointerExceptions mehr erzeugt und dabei das erwartete Optionsverhalten beibehält.

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
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
42/100

Neue Issues direkt in Ihr Postfach

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