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

[Enhancement] provider_nasdaq cache some data #6993

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

joshuaBri
Copy link
Contributor

I noticed that these interfaces return data very slowly. Could you please add caching to improve the performance?

@deeleeramone
Copy link
Contributor

deeleeramone commented Jan 3, 2025

Hi there,

The nasdaq calendars are slow requests because you need to request every date individually. Caching won't really help unless you repeatedly request the same data, and then if you are expecting new data from current events that would be excluded because it's already been cached.

What is your use case specifically that requires caching as opposed to your application storing results for recall later?

The historical_dividends response is fast and lightweight, there is not really a measurable gain to caching the request.

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.

2 participants