smarr / ReBenchDB

ReBenchDB records benchmark results and provides customizable reporting to track and analyze run-time performance of software programs.
MIT License
12 stars 6 forks source link

Add tests and a migration for the added expId in Trial’s unique constraint #116

Closed smarr closed 2 years ago

smarr commented 2 years ago

This completes the changes started in #115.

While just a theoretical issue, including the expId into the unique constraint prevents issues with unrelated experiments not being able to record their trial information.

This is in practice very unlikely to be an issue since the start time is part of the unique constraint. Though, for benchmarking it was noticed, and felt wrong not to fix it.