cmu-db / peloton / 5862

77%

push

travis-ci

Performance fixes: move all stats_mode checks to end of transaction (#1381)

* Removed all incremental stats updates from TimestampOrderingTransactionManager. Now processed entirely in TransactionManager's EndTransaction().

* Minor refactor on stats_type variable.

* Added more comments to EndTransaction()

* Adjust logic to find database oid in EndTransaction, also remove redundant code from Commit and Abort in TOTM.

* Removed stats_mode check from BeginTransaction(). This requires us to rely on the transaction timestamp to calculate the latency stat during EndTransaction(). Loss in precision looks reasonable for stats purposes:

transaction latency (new timestamp mode): 52.366000
transaction latency (old stats mode): 52.379732

Also moved all stats-related tasks from EndTransaction() to a dedicated function.

* clang-format-3.6 on modified lines.

43 of 43 new or added lines in 1 file covered. (100.0%)

24884 of 32151 relevant lines covered (77.4%)

79699.71 hits per line