offset x is not a multiple of limit y

Why is this check performed?

https://github.com/crnk-project/crnk-framework/blob/77fffa6023f49550852d6c55cfe7c27956a07a87/crnk-core/src/main/java/io/crnk/core/queryspec/pagingspec/OffsetLimitPagingBehavior.java#L99-L105

It’s throwing errors like:

offset 60 is not a multiple of limit 100

Even though this is a perfectly valid query:

SELECT * FROM tbl LIMIT 60,100;

/cc @mohlek

[weaselmetal]

Just because I realized why this check makes sense: It ensures compatibility between QueryParams and QuerySpec! Once the check is gone, you cannot correctly transform offset and limit into page size and number. So if someone has a setup with both parameter transporting objects, removing the check may cause issues. Just to keep that in mind.

[remmeier]

I guess @weaselmetal figured out the history. Once upon a time QueryParams was also still the main thing and QuerySpec the add-on on top of it.

Ideally the check gets deferred till it is actually needed. Support for number/size next to offset/limit is coming in the near future. And interoperability between the two would be quite important to not have to implement reusable repositories for both paging behaviors.