Skip to content

Commit

Permalink
Merge pull request #5280 from marcinczeczko/fix-for-5115
Browse files Browse the repository at this point in the history
Fix for #5115 - Do not fire DynamoDB recorder if no DynamoDB clients used
  • Loading branch information
gsmet authored Nov 7, 2019
2 parents 470b53e + d0f02d2 commit a6d464b
Showing 1 changed file with 8 additions and 6 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -162,14 +162,16 @@ private void checkClasspath(String className, String dependencyName) {
void buildClients(DynamodbClientBuildItem clientBuildItem, DynamodbRecorder recorder,
BeanContainerBuildItem beanContainer, ShutdownContextBuildItem shutdown) {

recorder.configureRuntimeConfig(config);
if (clientBuildItem.isCreateSyncClient() || clientBuildItem.isCreateAsyncClient()) {
recorder.configureRuntimeConfig(config);

if (clientBuildItem.isCreateSyncClient()) {
recorder.createClient(beanContainer.getValue(), shutdown);
}
if (clientBuildItem.isCreateSyncClient()) {
recorder.createClient(beanContainer.getValue(), shutdown);
}

if (clientBuildItem.isCreateAsyncClient()) {
recorder.createAsyncClient(beanContainer.getValue(), shutdown);
if (clientBuildItem.isCreateAsyncClient()) {
recorder.createAsyncClient(beanContainer.getValue(), shutdown);
}
}
}

Expand Down

0 comments on commit a6d464b

Please sign in to comment.