Open beikov opened 2 years ago
Queryable is returned instead of FullQueryBuilder was because of CTE queries (and I think also Set operations, but not sure if these can be paginated). I don't think just SelectNewBuilder & Queryable
is enough, because I think Pagination API's have been requested as well.
We can introduce a method that returns the FullQueryBuilder
and allow it to fail with a CastClassException for CTE/Set queries but thats going to be confusing.
As can be seen in this user question, it is important that we have an API that returns a
CriteriaBuilder
/FullQueryBuilder
, otherwise it's hard to use with entity views.Also see https://github.com/Blazebit/blaze-persistence/issues/1646, maybe it's worth returning a
SelectNewBuilder
instead? Or some subtype of it which also allows execution? Maybe we need a type that extendsSelectNewBuilder & Queryable
?