Default workers to 20?

[Rafnuss]

ecmwfr/R/wf_request_batch.R

Line 14 in bd7bf26

workers = 2,

Is there a reason for the default number to not be

workers = min(length(request_list, 20))

And should we add a warning if this number exceeds 20? Not sure what happen in this case.

[khufkens]

The reason for this is that although historically the theoretical limit was ~20 requests (old API, can't find the link to the docs), in practice this is often lower. The value is now set at 2 to make this a conscious decision by the user, rather than trigger a default greedy, take whatever we can get, position.

I should look at the code but if things get rejected there will be a retry, I think.

[khufkens]

Yep, the queue is N long and refilled based on another queue - the requests list. See while loop.

ecmwfr/R/wf_request_batch.R

Line 47 in bd7bf26

while (length(done) < length(request_list) & Sys.time() < timeout_time) {