apache / apache/parquet-java

ParquetFileWriter.appendFile does not close file handle - resource leak

Offen
#2,039 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Component: Java Component: Parquet Priority: Major Type: bug
Vorherrschende Sprache
Java
Sterne
3.1k
Forks
1.6k
Ø Merge
3 T. 12 Std.
Gemergte PRs (30 T.)
33

Beschreibung

Current implementation of the method ParquetFileWriter.appendFile does not close file handle.
```Java
ParquetFileReader.open(conf, file).appendTo(this);
```

My suggested fix is following:
```Java
try (ParquetFileReader reader = ParquetFileReader.open(conf, file)) {
reader.appendTo(this);
}
```

**Reporter**: [George Smith](https://issues.apache.org/jira/secure/ViewProfile.jspa?name=cabot)
#### PRs and other links:
- [GitHub Pull Request #708](https://github.com/apache/parquet-mr/pull/708)

**Note**: *This issue was originally created as [PARQUET-870](https://issues.apache.org/jira/browse/PARQUET-870). Please see the [migration documentation](https://issues.apache.org/jira/browse/PARQUET-2502) for further details.*

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Rechercherichtung

Beginne bei ParquetFileWriter.appendFile und untersuche, wie der von ParquetFileReader.open zurückgegebene ParquetFileReader verwaltet wird. Sieh dir Pull Request #708 auf die bereits vorhandenen Arbeiten an und verifiziere anschließend mithilfe der relevanten Projekttests, dass der Reader und sein Dateihandle geschlossen werden, nachdem appendFile abgeschlossen ist.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

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

Neue Issues direkt in Ihr Postfach

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