search

cmu-db / noisepage

Self-Driving Database Management System from Carnegie Mellon University
https://noise.page
MIT License
1.74k stars 502 forks source link

Remnants of Terrier: `TerrierTest` #1463

Open turingcompl33t opened 3 years ago

turingcompl33t commented 3 years ago

The very basic test harness defined in test_harness.h here is named TerrierTest. This should be refactored to NoisepageTest (or is it NoisePageTest?) to account for the name change.

Unfortunately this refactor would imply a relatively-large blast radius because this test harness is inherited from by quite a few of the individual test setups.

lmwnshn commented 3 years ago

If we're going there, we might also want to rename stuff like TerrierServer and a couple of stray CI/settings-related comments.

lmwnshn commented 3 years ago

Probably easier to do once the current "big" PRs are merged.

turingcompl33t commented 3 years ago

And not really all that important either, just came across this when looking at the new tests for the B+ tree.