Pull request here: https://github.com/infinispan/jdg/pull/113

Anistor: is there any reason why you copy-pasted the DSL tests instead of making the original test abstract and differing the test only in configuration? For example, the NonIndexedQueryDslConditionsTest has 1300 lines that are just duplicit.

Yes, the copied test class has some of the tests disabled because the non-indexed query is missbehaving in some edge cases and most notably because it does not support order by, at least not until this one gets integrated: https://github.com/infinispan/infinispan/pull/2655. The pre-existing tests are also a bit wrong in assuming a certain order of results without specifying 'order by'; the order was implicitly insured by the lucene backend, but not by non-indexed query so it leads to several failures.

So they are temporarily split until we fix all of them and make them pass. Then we can start unifying the suites for both indexed/non-indexed, embedded/remote.

I was also dissapointed some time ago seeing duplications like ClusteredQueryDslConditionsTest, but I accepted it since there was no quick solution at the moment.

OK, it seems that this needs some more work - I can't verify this as implemented and working when it's not.

Sorting and pagination is working now, https://bugzilla.redhat.com/show_bug.cgi?id=1112740 was resolved.

Moving target to CR2 so that we don't miss that.