Skip to content

Commit

Permalink
Skip the reroute if the whole batch was a noop
Browse files Browse the repository at this point in the history
  • Loading branch information
joegallo committed Jan 24, 2022
1 parent 0badf30 commit be4de6c
Showing 1 changed file with 4 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -83,8 +83,10 @@ public MetadataUpdateSettingsService(
builder.failure(task, e);
}
}
// reroute in case things change that require it (like number of replicas)
state = allocationService.reroute(state, "settings update");
if (state != currentState) {
// reroute in case things change that require it (like number of replicas)
state = allocationService.reroute(state, "settings update");
}
return builder.build(state);
};
}
Expand Down

0 comments on commit be4de6c

Please sign in to comment.