Hi,
Second thought on this.
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:
- keep the JDBCChacheLoader as it is
- also add the new impl (with a more expressive name, of course :)( perhaps deprecate the old one?)
- recommend the user to use the new class over the old one, highlighting performance improvements
Cheers,
Mircea
On 2/8/07, Manik Surtani <manik@jboss.org> wrote:
Guys,
I'm pretty keen on including Mircea's RedcdRecJDBCCacheLoader in 2.0.0:
1) Performance improvement based on removal of recursive DB calls
(uses linear calls instead - see Mircea's previous posts)
2) Backward compatible with the existing JDBC cache loader
3) No specific use case why the old JDBCCacheLoader should be used
instead of this.
So my question to you is this:
Do we drop the existing JDBCCacheLoader and rename the
RedcdRecJDBCCacheLoader (hate that name!) to JDBCCacheLoader?
Comments?
--
Manik Surtani
Lead, JBoss Cache
JBoss, a division of Red Hat
Email: manik@jboss.org
Telephone: +44 7786 702 706
MSN: manik@surtani.org
Yahoo/AIM/Skype: maniksurtani
_______________________________________________
jbosscache-dev mailing list
jbosscache-dev@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/jbosscache-dev