Skip to content

Commit

Permalink
fix: reflect model config in config.properties (#2382)
Browse files Browse the repository at this point in the history
* reflect the model config from properties

* honor model config in config.properties
  • Loading branch information
lxning authored Jun 12, 2023
1 parent 08a9093 commit 43d84ff
Showing 1 changed file with 21 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -278,6 +278,13 @@ private Model createModel(
archive.getModelVersion(),
Model.BATCH_SIZE,
batchSize);
} else {
batchSize =
configManager.getJsonIntValue(
archive.getModelName(),
archive.getModelVersion(),
Model.BATCH_SIZE,
batchSize);
}
model.setBatchSize(batchSize);

Expand All @@ -291,6 +298,13 @@ private Model createModel(
archive.getModelVersion(),
Model.MAX_BATCH_DELAY,
maxBatchDelay);
} else {
maxBatchDelay =
configManager.getJsonIntValue(
archive.getModelName(),
archive.getModelVersion(),
Model.MAX_BATCH_DELAY,
maxBatchDelay);
}
model.setMaxBatchDelay(maxBatchDelay);

Expand All @@ -304,6 +318,13 @@ private Model createModel(
archive.getModelVersion(),
Model.RESPONSE_TIMEOUT,
responseTimeout);
} else {
responseTimeout =
configManager.getJsonIntValue(
archive.getModelName(),
archive.getModelVersion(),
Model.RESPONSE_TIMEOUT,
responseTimeout);
}
model.setResponseTimeout(responseTimeout);
model.setWorkflowModel(isWorkflowModel);
Expand Down

0 comments on commit 43d84ff

Please sign in to comment.