svn commit: r1180096 - /james/server/trunk/queue-activemq/src/main/java/org/apache/james/queue/activemq/FileSystemBlobStrategy.java

2011-10-07 Thread norman
Author: norman Date: Fri Oct 7 16:42:36 2011 New Revision: 1180096 URL: http://svn.apache.org/viewvc?rev=1180096&view=rev Log: Make getFile(...) method of FileSystemBlobStrategy thread-safe. Thanks to Michael Herrmann for the patch. See JAMES-1327 Modified: james/server/trunk/queue-activem

Re: svn commit: r1180096 - /james/server/trunk/queue-activemq/src/main/java/org/apache/james/queue/activemq/FileSystemBlobStrategy.java

2011-10-08 Thread Felix Knecht
Wouldn't it make more sense to synchronize on queueF? There's BTW the same problem for deleteFile (see first part of diff). Regards Felix Index: src/main/java/org/apache/james/queue/activemq/FileSystemBlobStrategy.java === --- s

Re: svn commit: r1180096 - /james/server/trunk/queue-activemq/src/main/java/org/apache/james/queue/activemq/FileSystemBlobStrategy.java

2011-10-08 Thread Norman Maurer
I guess you are right synchronization should be faster then doing a fs access here anyway. Please commit :) Bye, Norman 2011/10/8 Felix Knecht : > Wouldn't it make more sense to synchronize on queueF? There's BTW the same > problem for deleteFile (see first part of diff). > > Regards > Feli