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

async_fetch 001: "Feat": edit function to be async for fetching data #1

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

Conversation

S0rushSotoudeh
Copy link

This type of task is time-consuming and would be better handled asynchronously. Before this change, the total fetch time was the sum of the fetch times for Bours and Fara Bourse. However, after this change, the total fetch time is now the maximum of the two times:
total fetch time=max(bourse fetch time,fara bourse fetch time)

This optimization can make the fetching process twice as fast as it was before.

For example:

ETA for Fara Bourse: 1.0372366905212402 seconds
ETA for Bours: 1.294360637664795 seconds
ETA for total: 1.294360637664795 seconds

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