As far as I can remember the only place we require query classes is in the check if it is enabled in the configuration builder.  All the query classes are loaded from the query-server module in infinispan-server without reflection already.  That is why I was saying if @tristantarrant had an opinion as I am fine with the change, but just wanted to make sure this is how it is normally done.

The main reason I was saying to keep the CL as empty as possible is to prevent further leaks of classes into core etc than shouldn't be there.