[Page File Management] 재즈(함석명) 미션 제출합니다. #5
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Chocochip101
requested changes
Sep 6, 2024
Comment on lines
+3
to
+8
| public enum PageType { | ||
|
|
||
| CLUSTERED_INDEX, | ||
| SECONDARY_INDEX, | ||
| UNDO_LOG; | ||
| } |
|
|
||
| @Override | ||
| protected boolean removeEldestEntry(Map.Entry<K, V> eldest) { | ||
| boolean isBufferPoolOverCapacity = size() > bufferSize; |
| public void writePage(PageId pageId, Page page) { | ||
| Path filePath = Paths.get(DIRECTORY_PATH, pageId.getFileName() + FILE_EXTENSION); | ||
|
|
||
| try (RandomAccessFile file = new RandomAccessFile(filePath.toString(), "rw")) { |
Comment on lines
+35
to
+44
| public void flush() { | ||
| for (Map.Entry<PageId, Page> entry : bufferPool.entrySet()) { | ||
| Page page = entry.getValue(); | ||
|
|
||
| if (!page.isPinned() && page.isDirty()) { | ||
| pageManager.savePage(entry.getKey(), page); | ||
| page.setDirty(false); | ||
| } | ||
| } | ||
| } |
| this.recordCount--; | ||
| } | ||
|
|
||
| public void setDirty(boolean isDirty) { |
| return false; | ||
| } | ||
|
|
||
| private void flushIfDirty(PageId pageId, Page page) { |
There was a problem hiding this comment.
Suggested change
| private void flushIfDirty(PageId pageId, Page page) { | |
| private void flush(PageId pageId, Page page) { |
해당 메서드를 사용하는 측에서는 더티 페이지의 여부가 중요하나요?
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
페이지 메타데이터와 페이지 객체를 정의하고 간단한 글로벌 버퍼 풀을 설계하였습니다.
엔진과 핸들러는 아직 존재하지 않으며, 추후에 엔진은 버퍼 풀을 통해 페이지를 관리하고 데이터를 조작할 수 있습니다.
에러 핸들링은 기능을 추가하면서 개선할 예정입니다.
구현 과정에서 페이지 구조를 학습하였고 페이지 헤더를 커스터마이징 하였습니다.
현재 구현에 필요한 페이지 번호(오프셋), 페이지 타입, 더티 페이지 여부, 레코드 수의 정보를 포함하였습니다.
포함되지 않은 부가 정보들은 필요 시에 추후 추가될 수 있습니다.
현재 버퍼 풀의 페이지 교체 전략을 LinkedHashMap을 통한 LRU(Least Recently Used) 알고리즘을 우선으로 하였는데
InnoDB 버퍼 풀을 참고했을 때 캐시 히트 확률을 높이는데 유리하고, 구현이 단순하여 빠르게 버퍼 풀을 구현하는데 유리하다고 판단하였습니다.