Skip to content

fix: correct MAX_PAGE_NUM calculation#644

Open
CactiFromDesert wants to merge 3 commits into
oceanbase:mainfrom
CactiFromDesert:fix-max-page-calculation
Open

fix: correct MAX_PAGE_NUM calculation#644
CactiFromDesert wants to merge 3 commits into
oceanbase:mainfrom
CactiFromDesert:fix-max-page-calculation

Conversation

@CactiFromDesert
Copy link
Copy Markdown

This PR addresses issue #563.

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

The changes compile successfully.

@CLAassistant
Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


CactiFromDesert seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@CactiFromDesert CactiFromDesert force-pushed the fix-max-page-calculation branch 3 times, most recently from 9026c90 to c8f29bb Compare May 18, 2026 11:52
@CactiFromDesert CactiFromDesert force-pushed the fix-max-page-calculation branch from c8f29bb to cbd49ad Compare May 18, 2026 11:55
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.

2 participants