----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/2632/ -----------------------------------------------------------
(Updated 2011-11-01 17:58:03.331152) Review request for wave, Michael MacFadd, vjrj, and soren. Changes ------- Addressed comments Summary ------- Improves persistence logic and addresses some old TODOs. Most of the patch was contributed by ([email protected]). -Removes the requirement for persistExecutor being single threaded. -Refines the persist() logic to make batch successive writes to storage. -Injects shared instance of persistExecutor instead of creating one instance per wavelet. Diffs (updated) ----- /src/org/waveprotocol/box/server/waveserver/DeltaStoreBasedWaveletState.java 1195229 /src/org/waveprotocol/box/server/waveserver/WaveServerModule.java 1195229 /test/org/waveprotocol/box/server/waveserver/WaveServerTest.java 1195229 Diff: https://reviews.apache.org/r/2632/diff Testing ------- yes. Thanks, Yuri
