[CLJ-858] Improve speed of STM by removing System.currentTimeMillis Created: 17/Oct/11 Updated: 22/Nov/13 Resolved: 22/Nov/13
|Affects Version/s:||Release 1.3|
|Fix Version/s:||Release 1.6|
Tested on Ubuntu and OSX
The inner class TVal in LockingTransaction and Ref has an unused milliseconds field (that is populated with System.currentTimeMillis()).
This patch removes the milliseconds from inner class TVal in LockingTransaction.java and Ref.java. Using a little test suite a increase of performance by up to 25% could be measured.
Screened by: Alex Miller
|Comment by Alex Miller [ 08/Aug/13 2:48 AM ]|
Patch applies cleanly. Author is a contributor.
Did not verify performance claims but I can't see how removing fields and calls to System.currentTimeMillis() could do anything but make STM usage smaller and faster.