apache / apache/parquet-java

Make PageReader AutoCloseable

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

Beschreibung

PageReader implementations can own resources — ColumnChunkPageReader owns decompression / decryption buffers — but the interface has no standard teardown method. Today ColumnChunkPageReadStore.close() calls a package-private releaseBuffers() on the concrete class. Alternative PageReader implementations have no idiomatic way to declare that they own resources.

Have PageReader extend AutoCloseable with a default no-op close(). Rename ColumnChunkPageReader.releaseBuffers() to close().

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Rechercherichtung

Beginne mit dem PageReader-Interface und untersuche anschließend ColumnChunkPageReader und ColumnChunkPageReadStore, insbesondere den bestehenden Aufruf von releaseBuffers(). Mache das Interface mit einem standardmäßigen Teardown ressourcensicher und stelle sicher, dass der konkrete Reader und Store die umbenannte close()-Methode verwenden; führe die relevanten Reader- und Page-Store-Tests aus, um zu bestätigen, dass die Ressourcenbereinigung weiterhin funktioniert.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
java
Bereich
data-engineering
Issue-Typ
Refactoring
Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Aktivitätsstatus
Aktiv
Klarheit
Klar beschrieben
Anfängerfreundlichkeit
82/100

Neue Issues direkt in Ihr Postfach

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