<div dir="ltr">+1 to apply it, but maybe we should use the occasion to move the error message to the Log interface as well.<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Oct 4, 2013 at 2:43 AM, Sanne Grinovero <span dir="ltr"><<a href="mailto:sanne@infinispan.org" target="_blank">sanne@infinispan.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Currently if a cache is configured with indexing enabled, but the<br>
Query module isn't on classpath, you get a simple warning.<br>
<br>
I think this should fail with a configuration validation error; it's<br>
not just safer but also consistent with many other validations.<br>
<br>
I've created ISPN-3583 and patch is ready.. any good reason to not apply it?<br>
<br>
Sanne<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>
</blockquote></div><br></div>