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>Mircea<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">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">manik@jboss.org
</a><br>Telephone: +44 7786 702 706<br>MSN: <a href="mailto:manik@surtani.org">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">jbosscache-dev@lists.jboss.org</a><br><a href="https://lists.jboss.org/mailman/listinfo/jbosscache-dev">https://lists.jboss.org/mailman/listinfo/jbosscache-dev</a><br></blockquote>
</div><br>