cmu-db / bustub

The BusTub Relational Database Management System (Educational)
https://15445.courses.cs.cmu.edu/
MIT License
3.92k stars 1.76k forks source link

fix(p2): fix wrong definition of function #701

Closed hnkjdaxzzq closed 1 month ago

hnkjdaxzzq commented 6 months ago

Modify the return value type of the ExtendibleHTableHeaderPage::GetDirectoryPageId(uint32_t directory_idx) const -> uint32_t function to page_id_t.

before: Using this function get pageid requires type conversion. After: The function return value conforms to the definition of the function name.

this function in bustub/src/storage/page/extendible_htable_header_page.cpp.

prashanthduvvada commented 1 month ago

Fixed with #649