<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0 Transitional//EN">
<HTML><HEAD>
<META http-equiv=Content-Type content="text/html; charset=us-ascii">
<META content="MSHTML 6.00.2900.2963" name=GENERATOR></HEAD>
<BODY
style="WORD-WRAP: break-word; khtml-nbsp-mode: space; khtml-line-break: after-white-space">
<DIV><SPAN class=687441214-23102006></SPAN><FONT face=Arial><FONT
color=#0000ff><FONT
size=2>Agreed. I wrote them as public static class Config, and forgot that <SPAN
class=687441214-23102006>for </SPAN>the MC <SPAN
class=687441214-23102006>we'</SPAN>d have to use the <SPAN
class=687441214-23102006>'</SPAN>$<SPAN class=687441214-23102006>' instead of
the '.' that's usable in java code</SPAN>.<SPAN class=687441214-23102006>
I did the -beans.xml files last night and didn't have the energy to deal with
it.</SPAN></FONT></FONT></FONT></DIV>
<DIV><FONT face=Arial><FONT color=#0000ff><FONT size=2><SPAN
class=687441214-23102006></SPAN></FONT></FONT></FONT> </DIV>
<DIV><FONT face=Arial><FONT color=#0000ff><FONT size=2><SPAN
class=687441214-23102006>The 2 files thing does suck
though.</SPAN></FONT></FONT></FONT></DIV>
<BLOCKQUOTE dir=ltr
style="PADDING-LEFT: 5px; MARGIN-LEFT: 5px; BORDER-LEFT: #0000ff 2px solid; MARGIN-RIGHT: 0px">
<DIV class=OutlookMessageHeader lang=en-us dir=ltr align=left>
<HR tabIndex=-1>
<FONT face=Tahoma size=2><B>From:</B> Manik Surtani [mailto:manik@jboss.org]
<BR><B>Sent:</B> Monday, October 23, 2006 5:17 AM<BR><B>To:</B> Brian
Stansberry<BR><B>Cc:</B> jbosscache-dev@lists.jboss.org<BR><B>Subject:</B> Re:
Pojo-style config<BR></FONT><BR></DIV>
<DIV></DIV>Looks great!
<DIV><BR class=khtml-block-placeholder></DIV>
<DIV>One Q though - about the specific config impls for CacheLoaders, etc., do
we really want them as inner classes,
e.g., org.jboss.cache.loader.FileCacheLoader$Config? From a Java
standpoint, this makes the most sense and is cleanest, can't say I like the
String representation of the inner class in XML though. :-) Just
me being a pedant, but isn't org.jboss.cache.loader.FileCacheLoaderConfig
better, sure this means 2 .java files per cache loader, but still?</DIV>
<DIV><BR class=khtml-block-placeholder></DIV>
<DIV>Cheers,<BR>
<DIV><SPAN class=Apple-style-span
style="WORD-SPACING: 0px; FONT: 12px Helvetica; TEXT-TRANSFORM: none; COLOR: rgb(0,0,0); TEXT-INDENT: 0px; WHITE-SPACE: normal; LETTER-SPACING: normal; BORDER-COLLAPSE: separate; border-spacing: 0px 0px; khtml-text-decorations-in-effect: none; apple-text-size-adjust: auto; orphans: 2; widows: 2"><SPAN
class=Apple-style-span
style="WORD-SPACING: 0px; FONT: 12px Helvetica; TEXT-TRANSFORM: none; COLOR: rgb(0,0,0); TEXT-INDENT: 0px; WHITE-SPACE: normal; LETTER-SPACING: normal; BORDER-COLLAPSE: separate; border-spacing: 0px 0px; khtml-text-decorations-in-effect: none; apple-text-size-adjust: auto; orphans: 2; widows: 2"><SPAN
class=Apple-style-span
style="WORD-SPACING: 0px; FONT: 12px Helvetica; TEXT-TRANSFORM: none; COLOR: rgb(0,0,0); TEXT-INDENT: 0px; WHITE-SPACE: normal; LETTER-SPACING: normal; BORDER-COLLAPSE: separate; border-spacing: 0px 0px; khtml-text-decorations-in-effect: none; apple-text-size-adjust: auto; orphans: 2; widows: 2">
<DIV>--</DIV>
<DIV>Manik Surtani</DIV>
<DIV><BR class=khtml-block-placeholder></DIV>
<DIV>Lead, JBoss Cache</DIV>
<DIV>JBoss, a division of Red Hat</DIV>
<DIV><BR class=khtml-block-placeholder></DIV>
<DIV>Email: <A href="mailto:manik@jboss.org">manik@jboss.org</A></DIV>
<DIV>Telephone: +44 7786 702 706</DIV>
<DIV>MSN: <A href="mailto:manik@surtani.org">manik@surtani.org</A></DIV>
<DIV>Yahoo/AIM/Skype: maniksurtani</DIV></SPAN></SPAN><BR
class=Apple-interchange-newline></SPAN></DIV><BR>
<DIV>
<DIV>On 23 Oct 2006, at 07:05, Brian Stansberry wrote:</DIV><BR
class=Apple-interchange-newline>
<BLOCKQUOTE type="cite">
<DIV style="MARGIN: 0px">FYI on where this sits:</DIV>
<DIV style="MIN-HEIGHT: 14px; MARGIN: 0px"><BR></DIV>
<DIV style="MARGIN: 0px">1) I checked in all the stuff we were discussing
last week.</DIV>
<DIV style="MARGIN: 0px">2) When I ran the testsuite, I didn't see
regressions.<SPAN class=Apple-converted-space> </SPAN>We'll see
what</DIV>
<DIV style="MARGIN: 0px">cc shows.<SPAN class=Apple-converted-space>
</SPAN>The existing testsuite actually excercises the new code</DIV>
<DIV style="MARGIN: 0px">fairly well, because even when the old style
Element and Properties</DIV>
<DIV style="MARGIN: 0px">configs are used, everything gets converted behind
the scenes to the new</DIV>
<DIV style="MARGIN: 0px">pojos.</DIV>
<DIV style="MARGIN: 0px">3) I created a stripped down AS head and created
-beans.xml files for</DIV>
<DIV style="MARGIN: 0px">the 4 JBC configs that will go in AS 5 -- tomcat,
ejb3 sfsb, ejb3</DIV>
<DIV style="MARGIN: 0px">entity, and the HAPartition cache.<SPAN
class=Apple-converted-space> </SPAN>They all deploy cleanly, so that's
a</DIV>
<DIV style="MARGIN: 0px">simple test of the MC-style config.<SPAN
class=Apple-converted-space> </SPAN>Don't know if they do what
they</DIV>
<DIV style="MARGIN: 0px">should (e.g. does eviction actually happen), but
I'm pretty confident</DIV>
<DIV style="MARGIN: 0px">about that.<SPAN class=Apple-converted-space>
</SPAN>The -beans.xml files are attached so you can see what they</DIV>
<DIV style="MARGIN: 0px">look like.</DIV>
<DIV style="MIN-HEIGHT: 14px; MARGIN: 0px"><BR></DIV>
<DIV style="MARGIN: 0px">There are details about that can be improved or at
least need</DIV>
<DIV style="MARGIN: 0px">discussing, but I think it's pretty reasonable for
an alpha.</DIV>
<DIV style="MIN-HEIGHT: 14px; MARGIN: 0px"><BR></DIV>
<DIV style="MARGIN: 0px">Cheers,</DIV>
<DIV style="MIN-HEIGHT: 14px; MARGIN: 0px"><BR></DIV>
<DIV style="MARGIN: 0px">Brian Stansberry</DIV>
<DIV style="MARGIN: 0px">Lead, AS Clustering</DIV>
<DIV style="MARGIN: 0px">JBoss, a division of Red Hat</DIV>
<DIV style="MARGIN: 0px">Ph: 510-396-3864</DIV>
<DIV style="MARGIN: 0px">skype: bstansberry<SPAN
class=Apple-converted-space> </SPAN><SPAN>
<DIV><tc6-cluster-beans.xml></DIV></SPAN><SPAN>
<DIV><cluster-beans.xml></DIV></SPAN><SPAN>
<DIV><ejb3-clustered-sfsbcache-beans.xml></DIV></SPAN><SPAN>
<DIV><ejb3-entity-cache-beans.xml></DIV></SPAN></DIV></BLOCKQUOTE></DIV><BR></DIV></BLOCKQUOTE></BODY></HTML>