Skip to content

Commit 78fcebc

Browse files
committed
Add annotation and rename method
1 parent ac703ce commit 78fcebc

File tree

2 files changed

+5
-3
lines changed

2 files changed

+5
-3
lines changed

server/src/main/java/org/elasticsearch/cluster/ProjectState.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,7 @@ public ClusterState updatedState(Consumer<ProjectMetadata.Builder> projectBuilde
8080
* Build a new {@link ClusterState} with the updated project.
8181
*/
8282
public ClusterState updatedState(ProjectMetadata updatedProject) {
83-
return ClusterState.builder(cluster).metadata(cluster.metadata().updateSingleProject(updatedProject)).build();
83+
return ClusterState.builder(cluster).metadata(cluster.metadata().withUpdatedProject(updatedProject)).build();
8484
}
8585

8686
/**

server/src/main/java/org/elasticsearch/cluster/metadata/Metadata.java

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -407,7 +407,9 @@ private Metadata updateSingleProject(Function<ProjectMetadata, ProjectMetadata>
407407
/**
408408
* Updates a single project in the metadata. This offers a more performant way of updating a single project compared to the Builder.
409409
*/
410-
public Metadata updateSingleProject(ProjectMetadata updatedProject) {
410+
@FixForMultiProject // We should reconsider whether this method is valuable once we update Metadata.Builder to hold constructed projects
411+
// instead of project builders.
412+
public Metadata withUpdatedProject(ProjectMetadata updatedProject) {
411413
final var existingProject = projectMetadata.get(updatedProject.id());
412414
if (existingProject == null) {
413415
throw new IllegalArgumentException(
@@ -1532,7 +1534,7 @@ public Metadata copyAndUpdateProject(ProjectId projectId, Consumer<ProjectMetada
15321534
}
15331535
final var builder = ProjectMetadata.builder(existingProject);
15341536
updater.accept(builder);
1535-
return updateSingleProject(builder.build());
1537+
return withUpdatedProject(builder.build());
15361538
}
15371539

15381540
public static class Builder {

0 commit comments

Comments
 (0)