https://issues.apache.org/bugzilla/show_bug.cgi?id=55612
--- Comment #5 from Luca Della Toffola <[email protected]> --- I attached two new patches. One used a static lock to synchronize the cache, the other a thread-local variables. Both now only uses a sub-set of the fields of the object, so we have simpler invalidation mechanism. In the same initial scenario (single-threaded) program both result in a similar performance improvement (thread-local version works slightly better) as the original patch. As in the other bug report I don't a have a testing scenario with multiple threads accessing the cache, in that case intuitively we will expect a performance degradation. (In reply to Yegor Kozlov from comment #1) > Can you please explain the logic in details? > > Why do you not compare lastFormat and format ? You cache lastFormat but use > it only in a "not null" clause and not in equals. > > > Why do you need to cache lastFormatSummary and collection of workbook's > format records? > > Yegor -- You are receiving this mail because: You are the assignee for the bug. --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
