This one time, at band camp, Stein M. Eliassen said:
SME>In org.castor.exolab.persist.LockEngine$TypeInfo.release you have this
SME>code-snippet:
SME> if ( entry.isDisposable() ) {
SME> cache.put( oid, entry );
SME> if ( entry.isExpired() ) {
SME> cache.expire(oid);
SME> entry.expired();
SME> }
SME> else
SME> cache.put( oid, entry );
SME> locks.remove( oid );
SME> }
SME>
SME>the entry is added twice to the cache and this is probably wrong, it is
SME>the first put that must be removed yes?
SME>
SME>The double-put isn't causing any harm other than to the performance...
Stein,
Good catch! My initial guess is the same as yours - that the first
put is not necessary. Comment it out and let me know the results.
Bruce
--
perl -e 'print unpack("u30","<0G)[EMAIL PROTECTED]&5R\"F9E<G)E=\$\!F<FEI+F-O;0\`\`");'
The Castor Project
http://www.castor.org/
Apache Geronimo
http://incubator.apache.org/projects/geronimo.html
-----------------------------------------------------------
If you wish to unsubscribe from this mailing, send mail to
[EMAIL PROTECTED] with a subject of:
unsubscribe castor-dev
