Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[15x] Remove "Future queries" findFutureList, findFutureIds, findFutureCount #3212

Merged
merged 1 commit into from
Aug 30, 2023

Conversation

rbygrave
Copy link
Member

Note that findFutureCount still exists as an internal implementation detail of findPagedList but not exposed via Query API for public use.

…reCount

Note that findFutureCount still exists as an internal implementation detail
of findPagedList but not exposed via Query API for public use.
@rbygrave rbygrave self-assigned this Aug 30, 2023
@rbygrave rbygrave merged commit 6f39efd into ebean-15x Aug 30, 2023
2 checks passed
@rob-bygrave rob-bygrave deleted the 15x/remove-future-queries branch September 4, 2023 04:24
@AntoineDuComptoirDesPharmacies

Hi @rbygrave,

What will be the replacement for this methods ? There will be some new methods to get entity list and count in parallel ?

Yours faithfully,
LCDP

@rbygrave
Copy link
Member Author

There is no planned replacement for these methods.

If you have applications that have found findFutureList and findFutureCount useful then you are encouraged to basically ask for them not to be removed.

@AntoineDuComptoirDesPharmacies

I see, at the moment we are using findFutureCount to get total count without blocking the main thread.
This is pretty useful to mirror the behavior of loadCount, getTotalCount from PagedList class.

Our workflow look like this :

  1. Run findFutureCount()
  2. Execute long process
  3. Return a result pairing the count + long process

Yours faithfully,
LCDP

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants