summaryrefslogtreecommitdiff
path: root/generate_app
diff options
context:
space:
mode:
authorMatthew Sackman <matthew@lshift.net>2009-07-19 16:47:14 +0100
committerMatthew Sackman <matthew@lshift.net>2009-07-19 16:47:14 +0100
commite3503c1672da06391ff10a301ed751931fd89407 (patch)
tree625d7c6a7141b26040a2ff5a9ae28aa58a2c52af /generate_app
parent1c3e228929a4828e8d731b63db05d31b4d66107d (diff)
downloadrabbitmq-server-git-e3503c1672da06391ff10a301ed751931fd89407.tar.gz
Fixed the commit bug. Really this should probably be in bug20470 but I really didn't want to have to deal with merging and the other information about this bug is in the above comments in 20980 so it's in here.
Now on commit, we test to see if we need to sync the current file. If so then we just store all the txn details in state for later dealing with. If not, we really do the commit there and then and reply. Interestingly, performance is actually better now than it was (see details in bug20470) but, eg, the one-in-one-out at altitude test has further reduced fsyncs from 21 to 6 and now completes in 2.1 seconds, not 3.6 (altitude of 1000, then 5000 @ one in, one out, then 1000 drain). All tests pass. We now guarantee that the messages will be fsync'd to disk _before_ anything is done to mnesia, in all cases of a txn_commit.
Diffstat (limited to 'generate_app')
0 files changed, 0 insertions, 0 deletions