<div dir="ltr">+1 to having a readable (and publicly advertised!) name for our default JGroups configuration. I particularly like the default-jgroups-tcp.xml proposal.<div><br></div><div>We always chastise users for going with their own configuration inherited from their Infinispan 4.0 days instead of starting with our default configuration and only modifying the settings that they really need. It would be much harder for the users to believe our advice if we call our configuration META-INF/we-dont-want-you-to-see-this/jgroups-tcp.xml</div>
<div><br></div><div>I vote to just log a warning on startup if we find look up the configuration file name in the classpath and find more than one match.</div><div><br></div><div>Bela: we also support absolute path names and URLs AFAIK but I don't really see a good use for those options.</div>
<div><br></div><div>Cheers</div><div>Dan</div><div><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Jun 13, 2014 at 10:09 AM, Alan Field <span dir="ltr"><<a href="mailto:afield@redhat.com" target="_blank">afield@redhat.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class=""><br>
<br>
----- Original Message -----<br>
> From: "Martin Gencur" <<a href="mailto:mgencur@redhat.com">mgencur@redhat.com</a>><br>
> To: "infinispan -Dev List" <<a href="mailto:infinispan-dev@lists.jboss.org">infinispan-dev@lists.jboss.org</a>><br>
</div><div class="">> Sent: Friday, June 13, 2014 7:36:10 AM<br>
> Subject: Re: [infinispan-dev] Issue with JGroups config files in ispn-core<br>
><br>
</div><div class="">> On 12.6.2014 20:14, Tristan Tarrant wrote:<br>
> > On 12/06/14 18:46, Dennis Reed wrote:<br>
> >> +1 to changing the name/directory.<br>
> >> -100 to changing the order of where it's looked for instead.<br>
> >><br>
> >> All resource lookups should use the normal rules for finding resources.<br>
> >> Don't change standard behavior without a *very* good reason.<br>
> >><br>
> >> Doing anything special (like META-INF/_internal/jgroups-udp.xml) is<br>
> >> completely non-intuitive<br>
> >> and will cause support issues down the road.<br>
> >> Using config/jgroups-udp.xml is standard, and would be immediately<br>
> >> understood by anyone.<br>
> >><br>
> > Users don't even need to know that META-INF/_internal/blah actually<br>
> > exists. It is just an internal detail when using the "default" (i.e.<br>
> > just enable clustering without explicitly specifying a configuration file).<br>
><br>
> My understanding was that users just take an example config. file (i.e.<br>
> jgroups-udp.xml), copy it into their application and modify. That's how<br>
> users get the same name for their configuration file as the default. So<br>
> in this case, they might find it again, even in<br>
> META-INF/_internal/jgroups.udp.xml :)<br>
<br>
</div>I agree that defining the order that configuration files are loaded is important and should be defined. I would prefer a more readable path like "META-INF/example_configurations/jgroups/udp.xml". This also gives us a good location to provide example cache configuration files as well. I also think that a message should be logged with the path to the configuration file being used:<br>
<br>
2014-06-12 06:45:02,871 [thread-name] INFO [package-name] Using JGroups configuration file 'jar:META-INF/example_configurations/jgroups/udp.xml'<br>
2014-06-12 06:45:02,871 [thread-name] INFO [package-name] Using JGroups configuration file 'file:/app_home/config/jgroups-udp.xml'<br>
<br>
This would make it more obvious to the user which configuration file is in use.<br>
<br>
Thanks,<br>
Alan<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
><br>
> Martin<br>
><br>
><br>
> ><br>
> > Tristan<br>
> > _______________________________________________<br>
> > infinispan-dev mailing list<br>
> > <a href="mailto:infinispan-dev@lists.jboss.org">infinispan-dev@lists.jboss.org</a><br>
> > <a href="https://lists.jboss.org/mailman/listinfo/infinispan-dev" target="_blank">https://lists.jboss.org/mailman/listinfo/infinispan-dev</a><br>
><br>
> _______________________________________________<br>
> infinispan-dev mailing list<br>
> <a href="mailto:infinispan-dev@lists.jboss.org">infinispan-dev@lists.jboss.org</a><br>
> <a href="https://lists.jboss.org/mailman/listinfo/infinispan-dev" target="_blank">https://lists.jboss.org/mailman/listinfo/infinispan-dev</a><br>
><br>
_______________________________________________<br>
infinispan-dev mailing list<br>
<a href="mailto:infinispan-dev@lists.jboss.org">infinispan-dev@lists.jboss.org</a><br>
<a href="https://lists.jboss.org/mailman/listinfo/infinispan-dev" target="_blank">https://lists.jboss.org/mailman/listinfo/infinispan-dev</a><br>
</div></div></blockquote></div><br></div>