Hi,<br><br>I've added the new JDBCCacheLoader impl on CVS. Old version is also present in the repository - JDBCCacheLoadeOld - deprecated class. I think it would be useful to keep it there for a while due to the following:
<br>- there is a CompatibilityTest test to ensure that the new one (class name JDBCCacheLoader) works fine on data created by the old one. Also any other further compatibility issues should be investigated here<br>- there is a benchmark code I've added which evaluates/compares performance. I think is might prove useful for further implementations or even in another benchmark
<br><br>There was an issue with the old impl: root was not persisted. The new one works fine on this.<br><br>Class I've added/modified:<br>JDBCCacheLoader<br>AdjListJDBCClassLoader<br>JDBCCacheLoaderOld<br>AdjListJDBCClassLoaderCompatibilityTest
<br>JDBCCacheLoader test<br>AdjListJDBCClassLoaderConfig <br>
JDBCClassLoaderConfig <br>tests\perf\org\jboss\cache\manualtests\cacheloader - the classes that perform the benchmarking/performance comparisons<br><br>Any feedback would be great!<br><br>Cheers,<br>Mircea<br><br><div><span class="gmail_quote">
On 2/8/07, <b class="gmail_sendername">Mircea Markus</b> <<a href="mailto:mircea.markus@gmail.com">mircea.markus@gmail.com</a>> wrote:</span><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
Hi,<br>
<br>
Second thought on this. <br>
My SoUglyNamedJDBCCacheLoader :) needs an additional configuration element - DBMS string concatenation function syntax. Normally concat(1,2) should make it for most of DBMs (works on mysql, oracle, mssql), but the fact is that this additional required config does not make the new impl 100% backward(you can not take the new class and put it in the place of old class). So I suggest following:
<br>- keep the JDBCChacheLoader as it is<br>- also add the new impl (with a more expressive name, of course :)( perhaps deprecate the old one?)<br>- recommend the user to use the new class over the old one, highlighting performance improvements
<br><br>Cheers,<br><span class="sg">Mircea</span><div><span class="e" id="q_110a2d4e72fb4988_2"><br><br><br> <div><span class="gmail_quote">On 2/8/07, <b class="gmail_sendername">Manik Surtani</b> <<a href="mailto:manik@jboss.org" target="_blank" onclick="return top.js.OpenExtLink(window,event,this)">
manik@jboss.org</a>> wrote:</span><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
Guys,<br><br>I'm pretty keen on including Mircea's RedcdRecJDBCCacheLoader in 2.0.0:<br><br>1) Performance improvement based on removal of recursive DB calls<br>(uses linear calls instead - see Mircea's previous posts)
<br>2) Backward compatible with the existing JDBC cache loader<br>3) No specific use case why the old JDBCCacheLoader should be used<br>instead of this.<br><br>So my question to you is this:<br><br>Do we drop the existing JDBCCacheLoader and rename the
<br>RedcdRecJDBCCacheLoader (hate that name!) to JDBCCacheLoader?<br><br>Comments?<br>--<br>Manik Surtani<br><br>Lead, JBoss Cache<br>JBoss, a division of Red Hat<br><br>Email: <a href="mailto:manik@jboss.org" target="_blank" onclick="return top.js.OpenExtLink(window,event,this)">
manik@jboss.org
</a><br>Telephone: +44 7786 702 706<br>MSN: <a href="mailto:manik@surtani.org" target="_blank" onclick="return top.js.OpenExtLink(window,event,this)">manik@surtani.org</a><br>Yahoo/AIM/Skype: maniksurtani<br><br><br><br>
_______________________________________________<br>jbosscache-dev mailing list
<br><a href="mailto:jbosscache-dev@lists.jboss.org" target="_blank" onclick="return top.js.OpenExtLink(window,event,this)">jbosscache-dev@lists.jboss.org</a><br><a href="https://lists.jboss.org/mailman/listinfo/jbosscache-dev" target="_blank" onclick="return top.js.OpenExtLink(window,event,this)">
https://lists.jboss.org/mailman/listinfo/jbosscache-dev</a><br></blockquote>
</div><br>
</span></div></blockquote></div><br>