Skip to content

Comments

Unpaged findAll()#25

Open
v-ladynev wants to merge 2 commits intopramoth:masterfrom
v-ladynev:unpaged-find-all-16
Open

Unpaged findAll()#25
v-ladynev wants to merge 2 commits intopramoth:masterfrom
v-ladynev:unpaged-find-all-16

Conversation

@v-ladynev
Copy link

Added two unpaged findAll methods

<R> List<R> findAll(Specification<T> spec, Class<R> projectionClass);

<R> List<R> findAll(Specification<T> spec, Class<R> projectionClass, Sort sort);

The first method can be emulated by using an existing paged findAll() method with Pageable.unpaged(). But it is not an obvious solution.

The second method much more complex to emulate, because it needs to create a new class that supports unpaged behavior with ability to provide sorting.

This pull request was inspired by @sic-sic changes. Many thanks to him.

Fixes #16

@v-ladynev v-ladynev changed the title Unpaged find all Unpaged findAll() Apr 5, 2022
@v-ladynev v-ladynev closed this by deleting the head repository Sep 24, 2022
@v-ladynev v-ladynev reopened this Sep 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

add a new findAll which takes only the Specification and return a List

1 participant