Refactor asynchronous client #107

ekouts · 2024-05-28T11:32:25Z

The asynchornous client has been refactored, with a small addition: the merging of requests needs to be enabled manually.

Changes:

Add timers in the logs for the asynchronous requests.
Locks are kept only for the merging of requests.
Merging of requests will not be attempted if the time between calls of the microservice is 0.
Merging of requests will not be attempted for /compute/acct when the user is not specifying the jobs IDs or when other parameters are passed (for example page_number or start_time).

Will close #94 .

…async_merge_reqs

ekouts added 8 commits April 29, 2024 14:11

wip

f598e05

wip

34b1e38

Merge branch 'main' of github.com:eth-cscs/pyfirecrest into refactor/…

96292a5

…async_merge_reqs

Add timers in async client

76d210b

Stash

8314c84

Don't merge reqs by default

01c5048

Bring back the old merge_request

28018d1

Fix internal dictionaries keys

a4019e2

ekouts requested a review from rsarm May 28, 2024 11:32

ekouts self-assigned this May 28, 2024

ekouts changed the title ~~Refactor async merge reqs add timers of requests in the~~ Refactor asynchronous client May 28, 2024

Move decorator to wrapper GET function

375bd81

ekouts mentioned this pull request May 29, 2024

AsyncFirecrest: Practicality and Efficiency #94

Closed

ekouts added 2 commits June 3, 2024 09:22

Merge branch 'main' of github.com:eth-cscs/pyfirecrest into refactor/…

3951eee

…async_merge_reqs

Merge branch 'main' of github.com:eth-cscs/pyfirecrest into refactor/…

4a4b87d

…async_merge_reqs

rsarm approved these changes Jun 13, 2024

View reviewed changes

ekouts merged commit 4537886 into eth-cscs:main Jun 13, 2024
8 checks passed

ekouts deleted the refactor/async_merge_reqs branch June 13, 2024 06:53

Provide feedback