You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Apr 1, 2023. It is now read-only.
Detail Description:
The original CachedClock achieves fast ticking by simply increasing the timestamp by 1 unit of time when now() is called. When precise time comparison is needed, we need to set refresh to true to get the time from std::chrono::steady_clock. This is causing issue in the 3SITxnTest. When it is calling the doRead, if there is a conflict, it will be resolved using the push operation (K23SIPartitionModule::_doPush) in Module.cpp. This is taking in a fast deadline as a parameter. If not synced properly, this deadline will be way behind the actual time and causing other deadlines to break.
We need to measure the performance impact of the CachedClock against steady_clock. If there is no significant impact, we should remove the CachedClock.
The text was updated successfully, but these errors were encountered:
Sign up for freeto subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Detail Description:
The original
CachedClock
achieves fast ticking by simply increasing the timestamp by 1 unit of time when now() is called. When precise time comparison is needed, we need to set refresh to true to get the time fromstd::chrono::steady_clock
. This is causing issue in the3SITxnTest
. When it is calling thedoRead
, if there is a conflict, it will be resolved using the push operation (K23SIPartitionModule::_doPush) inModule.cpp
. This is taking in a fast deadline as a parameter. If not synced properly, this deadline will be way behind the actual time and causing other deadlines to break.We need to measure the performance impact of the CachedClock against steady_clock. If there is no significant impact, we should remove the
CachedClock
.The text was updated successfully, but these errors were encountered: