cmu-db / bustub

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

fix(test): ensure compatibility std::uniform_int_distribution for binary data generation & improve overall format #587

Closed xzhseh closed 1 year ago

xzhseh commented 1 year ago

This PR solves #585 by changing the instantiation of std::uniform_int_distribution from the implicit char to the explicit int, which makes the static_assert in MacOSX13.3.sdk happy. (Detailed reason is mentioned in the issue) Also improve the overall format for this test🥰