apache / apache/parquet-java

ParquetFileWriter.appendFile does not close file handle - resource leak

Đang mở
#2,039 0 bình luận 0 reaction 0 người được giao Xem trên GitHub
Component: Java Component: Parquet Priority: Major Type: bug
Ngôn ngữ chính
Java
Star
3.1k
Fork
1.6k
Merge trung bình
3 ngày 12 giờ
Pull request đã merge (30 ngày)
33

Mô tả

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

Hướng dẫn đóng góp

Chưa lập chỉ mục được hướng dẫn đóng góp cho kho mã nguồn này

Hướng nghiên cứu

Bắt đầu tại ParquetFileWriter.appendFile và kiểm tra cách ParquetFileReader được ParquetFileReader.open trả về được quản lý. Xem lại Pull Request #708 để nắm phần công việc hiện có, sau đó xác minh bằng các test liên quan của dự án rằng reader và file handle của nó được đóng sau khi appendFile hoàn tất.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Đánh giá

Công nghệ
java
Lĩnh vực
data-engineering
Loại issue
Lỗi
Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức độ hoạt động
Đình trệ
Độ rõ ràng
Đặc tả rõ ràng
Mức phù hợp với người mới
25/100

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.