apache / apache/parquet-java

ParquetFileWriter.appendFile does not close file handle - resource leak

Abierto
#2,039 0 comentarios 0 reacciones 0 asignados Ver en GitHub
Component: Java Component: Parquet Priority: Major Type: bug
Lenguaje dominante
Java
Estrellas
3.1k
Forks
1.6k
Merge medio
3 d 12 h
PR fusionados (30 d)
33

Descripción

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.*

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Línea de trabajo

Comienza en ParquetFileWriter.appendFile e inspecciona cómo se gestiona el ParquetFileReader devuelto por ParquetFileReader.open. Revisa Pull Request #708 para consultar el trabajo existente y, a continuación, verifica mediante las pruebas relevantes del proyecto que el reader y su manejador de archivo se cierren después de que appendFile finalice.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
java
Área
data-engineering
Tipo de issue
Error
Dificultad
2/5
Tiempo estimado
1-3 horas
Estado de actividad
Estancado
Claridad
Bien especificado
Aptitud para principiantes
25/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.