Skip to content

Commit

Permalink
Fix queueSize setting is not honoured by AWS Batch executor (#3093)
Browse files Browse the repository at this point in the history
Signed-off-by: Ben Sherman <bentshermann@gmail.com>
  • Loading branch information
bentsherman committed Aug 4, 2022
1 parent 7e1c068 commit d07bb52
Showing 1 changed file with 5 additions and 2 deletions.
Expand Up @@ -185,12 +185,14 @@ class AwsBatchExecutor extends Executor implements ExtensionPoint {

final pollInterval = session.getPollInterval(name, Duration.of('10 sec'))
final dumpInterval = session.getMonitorDumpInterval(name)
final capacity = session.getQueueSize(name, 1000)

final def params = [
name: name,
session: session,
pollInterval: pollInterval,
dumpInterval: dumpInterval
dumpInterval: dumpInterval,
capacity: capacity
]

log.debug "Creating parallel monitor for executor '$name' > pollInterval=$pollInterval; dumpInterval=$dumpInterval"
Expand All @@ -217,7 +219,8 @@ class AwsBatchExecutor extends Executor implements ExtensionPoint {
*/
private ThrottlingExecutor createExecutorService(String name) {

final qs = session.getQueueSize(name, 5_000)
// queue size can be overridden by submitter options below
final qs = 5_000
final limit = session.getExecConfigProp(name,'submitRateLimit','50/s') as String
final size = Runtime.runtime.availableProcessors() * 5

Expand Down

0 comments on commit d07bb52

Please sign in to comment.