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
I made some old skool timing changes to the journal code (journal.patch) to determine where the time during update was being spent. Results from several updates are below. The TL;DR is that of the time spent in Journal.Update, about half of this is spent re-reading the data back to make sure that we wrote it out correctly.
If we removed this validation code, it would make updating a checkpoint in a witness about 20% faster. I'm raising this bug to discuss if we can now remove this validation code.
I made some old skool timing changes to the journal code (journal.patch) to determine where the time during update was being spent. Results from several updates are below. The TL;DR is that of the time spent in
Journal.Update
, about half of this is spent re-reading the data back to make sure that we wrote it out correctly.If we removed this validation code, it would make updating a checkpoint in a witness about 20% faster. I'm raising this bug to discuss if we can now remove this validation code.
The text was updated successfully, but these errors were encountered: