Hi,<div><br></div><div>Yes, I can adjust the patch next days. I&#39;ve just noticed that I send summary in not friendly format :), better one is now attached.</div><div><br></div><div>There is explanation for yours questions below.</div>
<div> <br><br><div class="gmail_quote">2009/8/23 Emmanuel Bernard <span dir="ltr">&lt;<a href="mailto:emmanuel@hibernate.org" target="_blank">emmanuel@hibernate.org</a>&gt;</span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div style="word-wrap:break-word">Hey Lukasz,<div>Your patch looks quite good and pass tests on my side.</div><div><br></div><div>I encourage others to check out the patch before we apply it (ideally another person form HSearch and one person from infinispan.</div>

<div><br></div><div><br></div><div>Lukasz, I have a few questions/remarks though before applying it. Can you answer / adjust the patch?</div><div><br></div><div>IndexWriterSetting</div><div><div>Why move to return Object in parsing from the initial int?</div>

<div></div></div></div></blockquote><div><br></div><div>IndexWriterSetting has to set up MergeScheduler in IndexWriter. Before, parsing was responsible for number conversion from String to int. Now I have to parse class name, and build/return MergeScheduler from it.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><div><div><br></div><div>Move DPHelper#createInfinispanCacheManager to IDP</div>
<div>this is not something that can be shared as it creates a hard dependency on infinispan otherwise.</div><div><br></div><div>in createInfinispanCacheManager</div>
<div>Don&#39;t log in error the fact that xml is not used if a default config is used. Just log in trace at best.</div><div><br></div><div>Rename InfinispanCacheManagerConfigurationImpl to DefaultInfinispanCacheManagerConfiguration or even better with a name describing nicely the behavior of the infinispan config.</div>

<div><br></div><div>in InfinispanIndexOutput, is it possible to get writeBytes bigger than buffer size? If yes, does newCheck creates the appropriate numbers of chunks?</div></div></div></blockquote><div><br></div><div>Yes it is possible. Writing process is divided into stages, during every stage can be written max: buffer_size bytes. At the end of the stage its checked if necessary is new chunk, if so new chunk is created.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><div><div></div><div><br></div><div>InfinispanDirectoryProvider</div>
<div>put the configuration proeprties available in the InfinispanDirectoryProvider javadoc.</div><div><br></div><div>I think the default cache name should be &quot;Hibernate Search&quot; instead of &quot;HSInfinispanCache&quot;. We know it&#39;s in infinispan :)</div>

<div><br></div><div>what&#39;s the try catch opening and closing an IW about? It looks weird.</div></div></div></blockquote><div><br></div><div>IW is opened with create=true parameter, first index have to be initialized/created. Always next IW is opened with create=false parameter, then data is appended to exisitng index. Similar things are done in other DP&#39;s.</div>
<div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><div><div>in stop()</div><div>you don&#39;t close the CacheManager? How is that?</div>
</div></div></blockquote><div><br></div><div>Yes. Should be closed.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word">
<div><div></div><div><br></div><div>InfinispanCacheManagerConfigurationImpl</div>
<div>What does &quot;Infinispan-Cluster&quot; correspond to? Why this name? Shouldn&#39;t it be &quot;Hibernate Search cluster&quot;?</div><div>Is it safe to override the GlobalConfiguration? What if JBoss AS use infinispan to run?</div>
</div></div></blockquote><div><br></div><div>This name is used to distinguish cluster used by HSearch - All nodes with the same name form a group. Yes, rather &quot;Hibernate Search cluster&quot; is better name. It is safe to modify GlobalConfiguration, there can be set up configuration for CacheManager like communication way (JGroups or something else), stack configuration for JGroups, etc.; where Configuration is used to configure specific cache. I think just the infinispan cluster name on JBoss AS have to be different from HSearch, then they will be independent.   </div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><div><div></div>
<div><br></div><div><div>Why the use of DummyTransactionManagerLookup. Doesn&#39;t Infinispan guess the right TM depending on the environment? e in JBoss As use the JBoss one etc?  I think GenericTransactionManagerLookup does that.</div>
</div></div></div></blockquote><div><br></div><div>Yes right, I was testing it with DummyTM and forgot to change it later.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div style="word-wrap:break-word"><div><div><div></div>
<div><br></div></div><div>InfinispanCacheManagerConfiguration</div><div>some javadoc on the methods would be useful. I don&#39;t know what do implement here.</div><div><br></div><div><br></div><div>Is there a better name for Metadata? Like FileMetadata maybe?</div>
</div></div></blockquote><div><br></div><div>Better FileMetadata or maybe FileHeader.</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word">
<div><div></div>
<div><br></div><div>Where is ispn-cache-default-conf.xml used? For tests only? If not: is it possible to use a programmatic version instead and what is &quot;It&#39;s a movie cache&quot;?</div></div></div></blockquote><div>
<br></div><div>Yes in tests only so far. However it can be used as a provided default configuration. I will send maybe question to infinispan group about best configuration parameters. &quot;It&#39;s a movie cache&quot; it&#39;s the name for cache configured in ispn-cache-default-conf.xml. In tests in this cache are stored indexes for entity Movie. Indexes for all other entities are stored in default HSInfinispanCache.</div>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><div><div></div><div><br></div><div>Emmanuel</div>
<div><br></div></div><div><br><div><br><div>Begin forwarded message:</div><br><blockquote type="cite"><div><div style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0px"><font face="Helvetica" size="3" color="#000000" style="font:12.0px Helvetica;color:#000000"><b>From: </b></font><font face="Helvetica" size="3" style="font:12.0px Helvetica">Łukasz Moreń &lt;<a href="mailto:lukasz.moren@gmail.com" target="_blank">lukasz.moren@gmail.com</a>&gt;</font></div>

<div style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0px"><font face="Helvetica" size="3" color="#000000" style="font:12.0px Helvetica;color:#000000"><b>Date: </b></font><font face="Helvetica" size="3" style="font:12.0px Helvetica">21 août 2009 02:11:03 HAEC</font></div>

<div style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0px"><font face="Helvetica" size="3" color="#000000" style="font:12.0px Helvetica;color:#000000"><b>To: </b></font><font face="Helvetica" size="3" style="font:12.0px Helvetica">Emmanuel Bernard &lt;<a href="mailto:emmanuel@hibernate.org" target="_blank">emmanuel@hibernate.org</a>&gt;</font></div>

<div style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0px"><font face="Helvetica" size="3" color="#000000" style="font:12.0px Helvetica;color:#000000"><b>Subject: </b></font><font face="Helvetica" size="3" style="font:12.0px Helvetica"><b>GSoC patch with Infinispan Directory Provider</b></font></div>

<div style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0px;min-height:14px"><br></div> </div><div>I&#39;m sending patch and piece of documentation - not much but necessary information are included. </div>

<div>There are some todos but I didn&#39;t manage to finish it yet.</div><div>I changed maven jgroups dependency to 2.8.beta2, before version was clashed with used by infinispan.</div> <div>In pom file there was dependency on hibernate common annotations 3.2.shapshot. It should&#39;t be 3.5?</div>

<div><br></div>Cheers,<div>Lukasz</div> </blockquote></div></div></div><br><div style="word-wrap:break-word"><div><div><blockquote type="cite"></blockquote></div></div></div><br><div style="word-wrap:break-word"><div><div>

<blockquote type="cite"></blockquote></div><br></div></div><br></blockquote></div><br></div>