Skip to content

Commit f5bb750

Browse files
committed
fix naming bug
1 parent f12d3d8 commit f5bb750

File tree

4 files changed

+19
-11
lines changed

4 files changed

+19
-11
lines changed

server/src/main/java/org/elasticsearch/TransportVersionSet.java

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -85,7 +85,7 @@ private static Map<String, TransportVersionSet> loadTransportVersionSets() {
8585
String versionLocation = "transport/" + name;
8686
try (InputStream inputStream = TransportVersionSet.class.getResourceAsStream(versionLocation)) {
8787
TransportVersionSet transportVersionSet = TransportVersionSet.fromXContent(inputStream, latestId);
88-
transportVersionSets.put(name, transportVersionSet);
88+
transportVersionSets.put(transportVersionSet.name, transportVersionSet);
8989
} catch (IOException ioe) {
9090
throw new UncheckedIOException("transport version set file not found at [ " + versionLocation + "]", ioe);
9191
}
@@ -95,14 +95,12 @@ private static Map<String, TransportVersionSet> loadTransportVersionSets() {
9595
}
9696

9797
private static List<TransportVersion> collectTransportVersions() {
98-
List<TransportVersion> tranportVersions = new ArrayList<>();
98+
List<TransportVersion> transportVersions = new ArrayList<>();
9999
for (TransportVersionSet transportVersionSet : TRANSPORT_VERSION_SETS.values()) {
100-
for (TransportVersion transportVersion : transportVersionSet.versions) {
101-
tranportVersions.add(transportVersion);
102-
}
100+
transportVersions.addAll(transportVersionSet.versions);
103101
}
104-
tranportVersions.sort(TransportVersion::compareTo);
105-
return tranportVersions;
102+
transportVersions.sort(TransportVersion::compareTo);
103+
return transportVersions;
106104
}
107105

108106
private static TransportVersionSet fromXContent(InputStream inputStream, int maxTransportId) throws IOException {

server/src/main/java/org/elasticsearch/TransportVersions.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -326,8 +326,7 @@ static TransportVersion def(int id) {
326326
public static final TransportVersion ML_INFERENCE_COHERE_API_VERSION = def(9_110_0_00);
327327
public static final TransportVersion ESQL_PROFILE_INCLUDE_PLAN = def(9_111_0_00);
328328
public static final TransportVersion MAPPINGS_IN_DATA_STREAMS = def(9_112_0_00);
329-
// Below is the first version in 9.2 and NOT in 9.1.
330-
public static final TransportVersion PROJECT_STATE_REGISTRY_RECORDS_DELETIONS = def(9_113_0_00);
329+
331330
/*
332331
* STOP! READ THIS FIRST! No, really,
333332
* ____ _____ ___ ____ _ ____ _____ _ ____ _____ _ _ ___ ____ _____ ___ ____ ____ _____ _

server/src/main/java/org/elasticsearch/cluster/project/ProjectStateRegistry.java

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@
1010
package org.elasticsearch.cluster.project;
1111

1212
import org.elasticsearch.TransportVersion;
13+
import org.elasticsearch.TransportVersionSet;
1314
import org.elasticsearch.TransportVersions;
1415
import org.elasticsearch.cluster.AbstractNamedDiffable;
1516
import org.elasticsearch.cluster.ClusterState;
@@ -38,6 +39,10 @@ public class ProjectStateRegistry extends AbstractNamedDiffable<ClusterState.Cus
3839
public static final String TYPE = "projects_registry";
3940
public static final ProjectStateRegistry EMPTY = new ProjectStateRegistry(Collections.emptyMap(), Collections.emptySet(), 0);
4041

42+
public static final TransportVersionSet PROJECT_STATE_REGISTRY_RECORDS_DELETIONS = TransportVersionSet.get(
43+
"project-state-registry-records-deletions"
44+
);
45+
4146
private final Map<ProjectId, Settings> projectsSettings;
4247
// Projects that have been marked for deletion based on their file-based setting
4348
private final Set<ProjectId> projectsMarkedForDeletion;
@@ -46,7 +51,7 @@ public class ProjectStateRegistry extends AbstractNamedDiffable<ClusterState.Cus
4651

4752
public ProjectStateRegistry(StreamInput in) throws IOException {
4853
projectsSettings = in.readMap(ProjectId::readFrom, Settings::readSettingsFromStream);
49-
if (in.getTransportVersion().onOrAfter(TransportVersions.PROJECT_STATE_REGISTRY_RECORDS_DELETIONS)) {
54+
if (PROJECT_STATE_REGISTRY_RECORDS_DELETIONS.isCompatible(in.getTransportVersion())) {
5055
projectsMarkedForDeletion = in.readCollectionAsImmutableSet(ProjectId::readFrom);
5156
projectsMarkedForDeletionGeneration = in.readVLong();
5257
} else {
@@ -118,7 +123,7 @@ public TransportVersion getMinimalSupportedVersion() {
118123
@Override
119124
public void writeTo(StreamOutput out) throws IOException {
120125
out.writeMap(projectsSettings);
121-
if (out.getTransportVersion().onOrAfter(TransportVersions.PROJECT_STATE_REGISTRY_RECORDS_DELETIONS)) {
126+
if (PROJECT_STATE_REGISTRY_RECORDS_DELETIONS.isCompatible(out.getTransportVersion())) {
122127
out.writeCollection(projectsMarkedForDeletion);
123128
out.writeVLong(projectsMarkedForDeletionGeneration);
124129
} else {
Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
1+
{
2+
"name": "project-state-registry-records-deletions",
3+
"ids": [
4+
9113000
5+
]
6+
}

0 commit comments

Comments
 (0)