Bruce Snyder wrote:
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.


Working fine here, already upgraded the server.


Do you want a regular bugzilla-report?

Stein

----------------------------------------------------------- If you wish to unsubscribe from this mailing, send mail to
[EMAIL PROTECTED] with a subject of:
unsubscribe castor-dev




Reply via email to