|
|
|
I merged your changes in the 2.1.1 release file with the bug fix
Please check if the merges are correct in AbstractDiskPersistenceListener.java.patch_2.2-rc file. The patch looks good to me.
(I wanted to report one minor issue, since I thought that the fout.close() call in store() is redundant. But I just realised that the ObjectInputStream constructor can throw an exception, so it isn't) Using buffered in and output streams to increase persisting performance about 20%.
Groups are still serialized for backward reasons. | ||||||||||||||||||||||||||||||||||||||||||||||
There are 3 main changes:
- don't delete cache files before storing (not necessary IMHO since the file will be overwritten by the outputstream anyway)
- used buffered in and output streams
This gave me a performance gain of ca. 20% for removing 750 entries with each around 3 groups
- don't serialize groups, but write them as plain strings
This gave me another 20%
Caveat: the last change isn't compatible with group files written by old versions.