Re: [infinispan-dev] Warnings vs. Fail Fast

2013-10-04 Thread Dan Berindei
+1 to apply it, but maybe we should use the occasion to move the error message to the Log interface as well. On Fri, Oct 4, 2013 at 2:43 AM, Sanne Grinovero sa...@infinispan.orgwrote: Currently if a cache is configured with indexing enabled, but the Query module isn't on classpath, you get a

Re: [infinispan-dev] Warnings vs. Fail Fast

2013-10-04 Thread Adrian Nistor
+1 On 10/04/2013 02:43 AM, Sanne Grinovero wrote: Currently if a cache is configured with indexing enabled, but the Query module isn't on classpath, you get a simple warning. I think this should fail with a configuration validation error; it's not just safer but also consistent with many

[infinispan-dev] Warnings vs. Fail Fast

2013-10-03 Thread Sanne Grinovero
Currently if a cache is configured with indexing enabled, but the Query module isn't on classpath, you get a simple warning. I think this should fail with a configuration validation error; it's not just safer but also consistent with many other validations. I've created ISPN-3583 and patch is