Make PageReader AutoCloseable
- 主要言語
- Java
- スター
- 3.1k
- フォーク
- 1.6k
- 平均マージ
- 3日 12時間
- マージ済み PR(30日)
- 33
説明
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().
コントリビューションガイド
このリポジトリのコントリビューションガイドは索引されていません
調査の方向性
PageReader インターフェースから始め、次に ColumnChunkPageReader と ColumnChunkPageReadStore を調べます。特に、既存の releaseBuffers() 呼び出しを確認してください。デフォルトの teardown によってインターフェースをリソースセーフにし、具象 reader と store が名前変更後の close() メソッドを使用するようにします。関連する reader と page-store のテストを実行し、リソースのクリーンアップが引き続き機能することを確認してください。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- java
- 領域
- data-engineering
- issue の種類
- リファクタリング
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 活発さ
- 活発
- 明瞭さ
- 明確に書かれている
- 初心者へのやさしさ
- 82/100