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

configured retry-delay for MPI has no (or very minimal) effect #6

Open
pautri opened this issue Jul 29, 2019 · 2 comments
Open

configured retry-delay for MPI has no (or very minimal) effect #6

pautri opened this issue Jul 29, 2019 · 2 comments

Comments

@pautri
Copy link

pautri commented Jul 29, 2019

the currently configured retry-delay for the MPI provider does not seem to have an effect. Retry timestamps of the last run (before switching to the OLAC fallback after 5 failed attempts):

2019-07-25T18:33:01,080
2019-07-25T18:33:01,488
2019-07-25T18:33:01,892
2019-07-25T18:33:02,370
2019-07-25T18:33:03,076

If the configured values are in milliseconds, they are a rather small change compared to the average interval of 400-500 ms between requests.

@twagoo
Copy link
Member

twagoo commented Aug 1, 2019

Had a closer look; it seems that the right configuration makes it to the production instance and is partially applied: while default retry count is 2, the MPI provider gets 5 retries. However the retry-delay settings seems to be ignored.

Config line:

<config url="https://archive.mpi.nl/oai2" max-retry-count="5" retry-delay="10 30 60 360"/>

@menzowindhouwer
Copy link
Contributor

Retry delays where in milliseconds, so 10 30 60 360 was faily short. Changed now to seconds inline with the timeout configuration.

3f908aa

menzowindhouwer/oai-harvest-manager@9582300

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

No branches or pull requests

3 participants