Skip to content

fix: correct MAX_PAGE_NUM calculation#645

Open
CactiFromDesert wants to merge 1 commit into
oceanbase:mainfrom
CactiFromDesert:fix-max-page-calculation-v2
Open

fix: correct MAX_PAGE_NUM calculation#645
CactiFromDesert wants to merge 1 commit into
oceanbase:mainfrom
CactiFromDesert:fix-max-page-calculation-v2

Conversation

@CactiFromDesert
Copy link
Copy Markdown

This PR addresses the MAX_PAGE_NUM calculation issue.

Changes:

  • Added sizeof(buffer_pool_id) to MAX_PAGE_NUM calculation in disk_buffer_pool.h
  • Removed sizeof(PageNum) from BP_PAGE_DATA_SIZE calculation in page.h

Closes #563

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented May 18, 2026

CLA assistant check
All committers have signed the CLA.

@CactiFromDesert CactiFromDesert force-pushed the fix-max-page-calculation-v2 branch from a456235 to 9c89061 Compare May 18, 2026 12:12
@CactiFromDesert CactiFromDesert force-pushed the fix-max-page-calculation-v2 branch from 9c89061 to 9108308 Compare May 18, 2026 12:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Question about the calculation of BPFileHeader::MAX_PAGE_NUM and BP_PAGE_DATA_SIZE

2 participants