search

cmu-db / peloton

The Self-Driving Database Management System
http://pelotondb.io
Apache License 2.0
2.04k stars 623 forks source link

Increase sleep time for occasionally failing ReadOnlyTest. #1443

Closed mbutrovich closed 6 years ago

mbutrovich commented 6 years ago

This hopefully addresses #1427. I still don't fully understand why this is the only test that occasionally fails in this manner. In theory a lot of our tests that sleep for exactly 1 epoch duration and expect the epoch to have advanced are making a big assumption about thread scheduling. Most should probably relax their sleep time like this change.

tli2 commented 6 years ago

I am probably also going to restart the build on this multiple times before merging it to make sure it works

coveralls commented 6 years ago

Coverage Status

Coverage increased (+0.007%) to 76.53% when pulling b7b36382ebaf4aca4197c145ebcef60ed944436c on mbutrovich:serializable_txn_test into 898219f9fe6e1d2b6901ea3797caac5c93850213 on cmu-db:master.