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

Bottleneck in showing the search results #30

Closed
blokhin opened this issue Sep 2, 2021 · 2 comments
Closed

Bottleneck in showing the search results #30

blokhin opened this issue Sep 2, 2021 · 2 comments

Comments

@blokhin
Copy link
Member

blokhin commented Sep 2, 2021

It seems there's a bottleneck in showing the search results, by the slowest provider. Until it returns the results, no other much faster providers' results are shown. This is strange, since the Optimade client per se is asynchronous.

@PaulMaly
Copy link
Member

PaulMaly commented Sep 6, 2021

It's not really strange, because asynchronous is not Optimade client, but http requests to optimade APIs. As a consumer of Optimade client, we want to get a full list of providers/structures from the whole hierarchy of APIs. One of the reasons for that is because Optimade client shouldn't try to assume which type of consumer use it. Some types of consumers just may not support some progressive form of data recieving. In addition, such a task will require the implementation of some kind of reactivity, e.g. observables to ship the data partially when it's ready.

@blokhin
Copy link
Member Author

blokhin commented Oct 11, 2021

Done as soon as #33 is approved.

@blokhin blokhin closed this as completed Oct 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants