I was talking to Sanne on IRC and he seemed to think that config-samples/all.xml is a recommended configuration. It clearly it is not (i.e. what's the point of having REPEATABLE_READ but no transactions? or what do 99% users really care about the serialization.marshaller and serialization.version? ) > From what I gather, all.xml is just a configuration sample showing all config options in action, not necessarily the best combo. So, we probably need to do a better job at separating what's sample/recommended stuff, and other XML configs? Thoughts? -- Galder Zamarreño Sr. Software Engineer Infinispan, JBoss Cache _______________________________________________ infinispan-dev mailing list infinispan-dev(a)lists.jboss.org https://lists.jboss.org/mailman/listinfo/infinispan-dev

I looked for references to all.xml in the testsuite and there's a couple of tests that validate or do so other work with it. So, I think that all.xml should remain but moved to test/resources/configs and as you suggest, have a recommended file with more sensible configuration and other options commented out. How should name such file? sample.xml?

Maybe even name it tests-all.xml so our users never confuse it again. On 10-12-15 5:18 AM, Galder Zamarreño wrote: > I looked for references to all.xml in the testsuite and there's a couple of tests that validate or do so other work with it. > > So, I think that all.xml should remain but moved to test/resources/configs and as you suggest, have a recommended file with more sensible configuration and other options commented out. How should name such file? sample.xml? > > On Dec 14, 2010, at 5:25 PM, Manik Surtani wrote: > _______________________________________________ infinispan-dev mailing list infinispan-dev(a)lists.jboss.org https://lists.jboss.org/mailman/listinfo/infinispan-dev

I've cleaned up the config files. Could you guys please have a look at the new sample.xml and let me know what you think? Keep in mind the 4.2 and 5.0 samples are (slightly) different. 4.2: https://github.com/maniksurtani/infinispan/commit/b76c9a55c8612d70dbcec9b... 5.0: https://github.com/maniksurtani/infinispan/commit/846b75d1115c1ae0dff2d4c... Cheers Manik On 16 Dec 2010, at 10:17, Galder Zamarreño wrote: > Either that or move it to test/resources. I prefer to do the latter and just leave src/main/resources/config-samples/infinispan-sample.xml > > FYI: https://issues.jboss.org/browse/ISPN-837 > > On Dec 15, 2010, at 1:32 PM, Vladimir Blagojevic wrote: > >> Maybe even name it tests-all.xml so our users never confuse it again. >> >> On 10-12-15 5:18 AM, Galder Zamarreño wrote: >>> I looked for references to all.xml in the testsuite and there's a couple of tests that validate or do so other work with it. >>> >>> So, I think that all.xml should remain but moved to test/resources/configs and as you suggest, have a recommended file with more sensible configuration and other options commented out. How should name such file? sample.xml? >>> >>> On Dec 14, 2010, at 5:25 PM, Manik Surtani wrote: >>> >> >> _______________________________________________ >> infinispan-dev mailing list >> infinispan-dev(a)lists.jboss.org >> https://lists.jboss.org/mailman/listinfo/infinispan-dev > > -- > Galder Zamarreño > Sr. Software Engineer > Infinispan, JBoss Cache > > > _______________________________________________ > infinispan-dev mailing list > infinispan-dev(a)lists.jboss.org > https://lists.jboss.org/mailman/listinfo/infinispan-dev -- Manik Surtani manik(a)jboss.org twitter.com/maniksurtani Lead, Infinispan http://www.infinispan.org _______________________________________________ infinispan-dev mailing list infinispan-dev(a)lists.jboss.org https://lists.jboss.org/mailman/listinfo/infinispan-dev