Skip to content

Commit 25f4c8d

Browse files
jdconradjdconrad
committed
Migrate transport versions 9086_000 through 9007_000 (elastic#136221)
ES-12334
1 parent 31f6f83 commit 25f4c8d

File tree

78 files changed

+293
-189
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

78 files changed

+293
-189
lines changed

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

Lines changed: 0 additions & 32 deletions
Original file line numberDiff line numberDiff line change
@@ -140,38 +140,6 @@ static TransportVersion def(int id) {
140140
public static final TransportVersion TIMEOUT_GET_PARAM_FOR_RESOLVE_CLUSTER = def(8_838_0_00);
141141
public static final TransportVersion INFERENCE_REQUEST_ADAPTIVE_RATE_LIMITING = def(8_839_0_00);
142142
public static final TransportVersion ML_INFERENCE_IBM_WATSONX_RERANK_ADDED = def(8_840_0_00);
143-
public static final TransportVersion ESQL_PROFILE_ASYNC_NANOS = def(9_007_00_0);
144-
public static final TransportVersion ESQL_LOOKUP_JOIN_SOURCE_TEXT = def(9_008_0_00);
145-
public static final TransportVersion SLM_UNHEALTHY_IF_NO_SNAPSHOT_WITHIN = def(9_010_0_00);
146-
public static final TransportVersion REMOVE_REPOSITORY_CONFLICT_MESSAGE = def(9_012_0_00);
147-
public static final TransportVersion ESQL_SERIALIZE_SOURCE_FUNCTIONS_WARNINGS = def(9_016_0_00);
148-
public static final TransportVersion ESQL_DRIVER_NODE_DESCRIPTION = def(9_017_0_00);
149-
public static final TransportVersion MULTI_PROJECT = def(9_018_0_00);
150-
public static final TransportVersion UNASSIGENEDINFO_RESHARD_ADDED = def(9_022_0_00);
151-
public static final TransportVersion MAX_OPERATION_SIZE_REJECTIONS_ADDED = def(9_024_0_00);
152-
public static final TransportVersion ESQL_THREAD_NAME_IN_DRIVER_PROFILE = def(9_027_0_00);
153-
public static final TransportVersion INDEX_RESHARDING_METADATA = def(9_031_0_00);
154-
public static final TransportVersion INDEXING_STATS_INCLUDES_RECENT_WRITE_LOAD = def(9_034_0_00);
155-
public static final TransportVersion INDEX_METADATA_INCLUDES_RECENT_WRITE_LOAD = def(9_036_0_00);
156-
public static final TransportVersion PROJECT_ID_IN_SNAPSHOT = def(9_040_0_00);
157-
public static final TransportVersion INDEX_STATS_AND_METADATA_INCLUDE_PEAK_WRITE_LOAD = def(9_041_0_00);
158-
public static final TransportVersion ESQL_REMOVE_AGGREGATE_TYPE = def(9_045_0_00);
159-
public static final TransportVersion ADD_PROJECT_ID_TO_DSL_ERROR_INFO = def(9_046_0_00);
160-
public static final TransportVersion REPO_ANALYSIS_COPY_BLOB = def(9_048_0_00);
161-
public static final TransportVersion DEAD_ESQL_DOCUMENTS_FOUND_AND_VALUES_LOADED = def(9_052_0_00);
162-
public static final TransportVersion DEAD_BATCHED_QUERY_EXECUTION_DELAYABLE_WRITABLE = def(9_053_0_00);
163-
public static final TransportVersion BATCHED_QUERY_EXECUTION_DELAYABLE_WRITABLE = def(9_057_0_00);
164-
public static final TransportVersion COMPRESS_DELAYABLE_WRITEABLE = def(9_059_0_00);
165-
public static final TransportVersion SYNONYMS_REFRESH_PARAM = def(9_060_0_00);
166-
public static final TransportVersion DOC_FIELDS_AS_LIST = def(9_061_0_00);
167-
public static final TransportVersion DENSE_VECTOR_OFF_HEAP_STATS = def(9_062_00_0);
168-
public static final TransportVersion PROJECT_METADATA_SETTINGS = def(9_066_0_00);
169-
public static final TransportVersion WRITE_LOAD_INCLUDES_BUFFER_WRITES = def(9_070_0_00);
170-
public static final TransportVersion FILE_SETTINGS_HEALTH_INFO = def(9_072_0_00);
171-
public static final TransportVersion ESQL_FIELD_ATTRIBUTE_DROP_TYPE = def(9_075_0_00);
172-
public static final TransportVersion ESQL_TIME_SERIES_SOURCE_STATUS = def(9_076_0_00);
173-
public static final TransportVersion NODES_STATS_SUPPORTS_MULTI_PROJECT = def(9_079_0_00);
174-
public static final TransportVersion ESQL_LIMIT_ROW_SIZE = def(9_085_0_00);
175143

176144
/*
177145
* STOP! READ THIS FIRST! No, really,

‎server/src/main/java/org/elasticsearch/action/search/QueryPhaseResultConsumer.java‎

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@
1212
import org.apache.logging.log4j.LogManager;
1313
import org.apache.logging.log4j.Logger;
1414
import org.apache.lucene.search.TopDocs;
15-
import org.elasticsearch.TransportVersions;
15+
import org.elasticsearch.TransportVersion;
1616
import org.elasticsearch.action.search.SearchPhaseController.TopDocsStats;
1717
import org.elasticsearch.common.breaker.CircuitBreaker;
1818
import org.elasticsearch.common.breaker.CircuitBreakingException;
@@ -665,9 +665,13 @@ record MergeResult(
665665
long estimatedSize
666666
) implements Writeable {
667667

668+
private static final TransportVersion BATCHED_QUERY_EXECUTION_DELAYABLE_WRITEABLE = TransportVersion.fromName(
669+
"batched_query_execution_delayable_writeable"
670+
);
671+
668672
static MergeResult readFrom(StreamInput in) throws IOException {
669673
return new MergeResult(List.of(), Lucene.readTopDocsIncludingShardIndex(in), in.readOptionalWriteable(i -> {
670-
if (i.getTransportVersion().onOrAfter(TransportVersions.BATCHED_QUERY_EXECUTION_DELAYABLE_WRITABLE)) {
674+
if (i.getTransportVersion().supports(BATCHED_QUERY_EXECUTION_DELAYABLE_WRITEABLE)) {
671675
return DelayableWriteable.delayed(InternalAggregations::readFrom, i);
672676
} else {
673677
return DelayableWriteable.referencing(InternalAggregations.readFrom(i));
@@ -681,9 +685,7 @@ public void writeTo(StreamOutput out) throws IOException {
681685
out.writeOptionalWriteable(
682686
reducedAggs == null
683687
? null
684-
: (out.getTransportVersion().onOrAfter(TransportVersions.BATCHED_QUERY_EXECUTION_DELAYABLE_WRITABLE)
685-
? reducedAggs
686-
: reducedAggs.expand())
688+
: (out.getTransportVersion().supports(BATCHED_QUERY_EXECUTION_DELAYABLE_WRITEABLE) ? reducedAggs : reducedAggs.expand())
687689
);
688690
out.writeVLong(estimatedSize);
689691
}

‎server/src/main/java/org/elasticsearch/action/synonyms/DeleteSynonymRuleAction.java‎

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99

1010
package org.elasticsearch.action.synonyms;
1111

12-
import org.elasticsearch.TransportVersions;
12+
import org.elasticsearch.TransportVersion;
1313
import org.elasticsearch.action.ActionRequestValidationException;
1414
import org.elasticsearch.action.ActionType;
1515
import org.elasticsearch.action.LegacyActionRequest;
@@ -31,6 +31,9 @@ public DeleteSynonymRuleAction() {
3131
}
3232

3333
public static class Request extends LegacyActionRequest {
34+
35+
private static final TransportVersion SYNONYMS_REFRESH_PARAM = TransportVersion.fromName("synonyms_refresh_param");
36+
3437
private final String synonymsSetId;
3538
private final String synonymRuleId;
3639
private final boolean refresh;
@@ -39,7 +42,7 @@ public Request(StreamInput in) throws IOException {
3942
super(in);
4043
this.synonymsSetId = in.readString();
4144
this.synonymRuleId = in.readString();
42-
if (in.getTransportVersion().onOrAfter(TransportVersions.SYNONYMS_REFRESH_PARAM)) {
45+
if (in.getTransportVersion().supports(SYNONYMS_REFRESH_PARAM)) {
4346
this.refresh = in.readBoolean();
4447
} else {
4548
this.refresh = true;
@@ -70,7 +73,7 @@ public void writeTo(StreamOutput out) throws IOException {
7073
super.writeTo(out);
7174
out.writeString(synonymsSetId);
7275
out.writeString(synonymRuleId);
73-
if (out.getTransportVersion().onOrAfter(TransportVersions.SYNONYMS_REFRESH_PARAM)) {
76+
if (out.getTransportVersion().supports(SYNONYMS_REFRESH_PARAM)) {
7477
out.writeBoolean(refresh);
7578
}
7679
}

‎server/src/main/java/org/elasticsearch/action/synonyms/PutSynonymRuleAction.java‎

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99

1010
package org.elasticsearch.action.synonyms;
1111

12-
import org.elasticsearch.TransportVersions;
12+
import org.elasticsearch.TransportVersion;
1313
import org.elasticsearch.action.ActionRequestValidationException;
1414
import org.elasticsearch.action.ActionType;
1515
import org.elasticsearch.action.LegacyActionRequest;
@@ -40,6 +40,9 @@ public PutSynonymRuleAction() {
4040
}
4141

4242
public static class Request extends LegacyActionRequest {
43+
44+
private static final TransportVersion SYNONYMS_REFRESH_PARAM = TransportVersion.fromName("synonyms_refresh_param");
45+
4346
private final String synonymsSetId;
4447
private final SynonymRule synonymRule;
4548
private final boolean refresh;
@@ -59,7 +62,7 @@ public Request(StreamInput in) throws IOException {
5962
super(in);
6063
this.synonymsSetId = in.readString();
6164
this.synonymRule = new SynonymRule(in);
62-
if (in.getTransportVersion().onOrAfter(TransportVersions.SYNONYMS_REFRESH_PARAM)) {
65+
if (in.getTransportVersion().supports(SYNONYMS_REFRESH_PARAM)) {
6366
this.refresh = in.readBoolean();
6467
} else {
6568
this.refresh = true;
@@ -105,7 +108,7 @@ public void writeTo(StreamOutput out) throws IOException {
105108
super.writeTo(out);
106109
out.writeString(synonymsSetId);
107110
synonymRule.writeTo(out);
108-
if (out.getTransportVersion().onOrAfter(TransportVersions.SYNONYMS_REFRESH_PARAM)) {
111+
if (out.getTransportVersion().supports(SYNONYMS_REFRESH_PARAM)) {
109112
out.writeBoolean(refresh);
110113
}
111114
}

‎server/src/main/java/org/elasticsearch/action/synonyms/PutSynonymsAction.java‎

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99

1010
package org.elasticsearch.action.synonyms;
1111

12-
import org.elasticsearch.TransportVersions;
12+
import org.elasticsearch.TransportVersion;
1313
import org.elasticsearch.action.ActionRequestValidationException;
1414
import org.elasticsearch.action.ActionType;
1515
import org.elasticsearch.action.LegacyActionRequest;
@@ -57,11 +57,13 @@ public static class Request extends LegacyActionRequest {
5757
PARSER.declareObjectArray(ConstructingObjectParser.constructorArg(), (p, c) -> SynonymRule.fromXContent(p), SYNONYMS_SET_FIELD);
5858
}
5959

60+
private static final TransportVersion SYNONYMS_REFRESH_PARAM = TransportVersion.fromName("synonyms_refresh_param");
61+
6062
public Request(StreamInput in) throws IOException {
6163
super(in);
6264
this.synonymsSetId = in.readString();
6365
this.synonymRules = in.readArray(SynonymRule::new, SynonymRule[]::new);
64-
if (in.getTransportVersion().onOrAfter(TransportVersions.SYNONYMS_REFRESH_PARAM)) {
66+
if (in.getTransportVersion().supports(SYNONYMS_REFRESH_PARAM)) {
6567
this.refresh = in.readBoolean();
6668
} else {
6769
this.refresh = false;
@@ -104,7 +106,7 @@ public void writeTo(StreamOutput out) throws IOException {
104106
super.writeTo(out);
105107
out.writeString(synonymsSetId);
106108
out.writeArray(synonymRules);
107-
if (out.getTransportVersion().onOrAfter(TransportVersions.SYNONYMS_REFRESH_PARAM)) {
109+
if (out.getTransportVersion().supports(SYNONYMS_REFRESH_PARAM)) {
108110
out.writeBoolean(refresh);
109111
}
110112
}

‎server/src/main/java/org/elasticsearch/action/synonyms/SynonymUpdateResponse.java‎

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99

1010
package org.elasticsearch.action.synonyms;
1111

12-
import org.elasticsearch.TransportVersions;
12+
import org.elasticsearch.TransportVersion;
1313
import org.elasticsearch.action.ActionResponse;
1414
import org.elasticsearch.action.admin.indices.analyze.ReloadAnalyzersResponse;
1515
import org.elasticsearch.common.io.stream.StreamInput;
@@ -32,12 +32,14 @@ public class SynonymUpdateResponse extends ActionResponse implements ToXContentO
3232
public static final String RELOAD_ANALYZERS_DETAILS_FIELD = "reload_analyzers_details";
3333
static final ReloadAnalyzersResponse EMPTY_RELOAD_ANALYZER_RESPONSE = new ReloadAnalyzersResponse(0, 0, 0, List.of(), Map.of());
3434

35+
private static final TransportVersion SYNONYMS_REFRESH_PARAM = TransportVersion.fromName("synonyms_refresh_param");
36+
3537
private final UpdateSynonymsResultStatus updateStatus;
3638
private final ReloadAnalyzersResponse reloadAnalyzersResponse;
3739

3840
public SynonymUpdateResponse(StreamInput in) throws IOException {
3941
this.updateStatus = in.readEnum(UpdateSynonymsResultStatus.class);
40-
if (in.getTransportVersion().onOrAfter(TransportVersions.SYNONYMS_REFRESH_PARAM)) {
42+
if (in.getTransportVersion().supports(SYNONYMS_REFRESH_PARAM)) {
4143
this.reloadAnalyzersResponse = in.readOptionalWriteable(ReloadAnalyzersResponse::new);
4244
} else {
4345
this.reloadAnalyzersResponse = new ReloadAnalyzersResponse(in);
@@ -72,7 +74,7 @@ public XContentBuilder toXContent(XContentBuilder builder, Params params) throws
7274
@Override
7375
public void writeTo(StreamOutput out) throws IOException {
7476
out.writeEnum(updateStatus);
75-
if (out.getTransportVersion().onOrAfter(TransportVersions.SYNONYMS_REFRESH_PARAM)) {
77+
if (out.getTransportVersion().supports(SYNONYMS_REFRESH_PARAM)) {
7678
out.writeOptionalWriteable(reloadAnalyzersResponse);
7779
} else {
7880
if (reloadAnalyzersResponse == null) {

‎server/src/main/java/org/elasticsearch/cluster/ClusterState.java‎

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -159,6 +159,8 @@ public CompatibilityVersions read(StreamInput in, String key) throws IOException
159159

160160
public static final long UNKNOWN_VERSION = -1;
161161

162+
private static final TransportVersion MULTI_PROJECT = TransportVersion.fromName("multi_project");
163+
162164
/**
163165
* Monotonically increasing on (and therefore uniquely identifies) <i>committed</i> states. However sometimes a state is created/applied
164166
* without committing it, for instance to add a {@link NoMasterBlockService#getNoMasterBlock}.
@@ -1291,7 +1293,7 @@ public static ClusterState readFrom(StreamInput in, DiscoveryNode localNode) thr
12911293
builder.version = in.readLong();
12921294
builder.uuid = in.readString();
12931295
builder.metadata = Metadata.readFrom(in);
1294-
if (in.getTransportVersion().onOrAfter(TransportVersions.MULTI_PROJECT)) {
1296+
if (in.getTransportVersion().supports(MULTI_PROJECT)) {
12951297
builder.routingTable = GlobalRoutingTable.readFrom(in);
12961298
} else {
12971299
final RoutingTable rt = RoutingTable.readFrom(in);
@@ -1317,7 +1319,7 @@ public void writeTo(StreamOutput out) throws IOException {
13171319
out.writeLong(version);
13181320
out.writeString(stateUUID);
13191321
metadata.writeTo(out);
1320-
if (out.getTransportVersion().onOrAfter(TransportVersions.MULTI_PROJECT)) {
1322+
if (out.getTransportVersion().supports(MULTI_PROJECT)) {
13211323
routingTable.writeTo(out);
13221324
} else {
13231325
routingTable.getRoutingTable().writeTo(out);

‎server/src/main/java/org/elasticsearch/cluster/SnapshotsInProgress.java‎

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1775,6 +1775,8 @@ public void writeTo(StreamOutput out) throws IOException {
17751775

17761776
private static final class SnapshotInProgressDiff implements NamedDiff<Custom> {
17771777

1778+
private static final TransportVersion PROJECT_ID_IN_SNAPSHOT = TransportVersion.fromName("project_id_in_snapshot");
1779+
17781780
private final SnapshotsInProgress after;
17791781

17801782
private final DiffableUtils.MapDiff<ProjectRepo, ByRepo, Map<ProjectRepo, ByRepo>> mapDiff;
@@ -1787,7 +1789,7 @@ private static final class SnapshotInProgressDiff implements NamedDiff<Custom> {
17871789
}
17881790

17891791
SnapshotInProgressDiff(StreamInput in) throws IOException {
1790-
if (in.getTransportVersion().before(TransportVersions.PROJECT_ID_IN_SNAPSHOT)) {
1792+
if (in.getTransportVersion().supports(PROJECT_ID_IN_SNAPSHOT) == false) {
17911793
final var oldMapDiff = DiffableUtils.readJdkMapDiff(
17921794
in,
17931795
DiffableUtils.getStringKeySerializer(),
@@ -1838,13 +1840,13 @@ public String getWriteableName() {
18381840
public void writeTo(StreamOutput out) throws IOException {
18391841
assert after != null : "should only write instances that were diffed from this node's state";
18401842
if (out.getTransportVersion().onOrAfter(DIFFABLE_VERSION)) {
1841-
if (out.getTransportVersion().before(TransportVersions.PROJECT_ID_IN_SNAPSHOT)) {
1843+
if (out.getTransportVersion().supports(PROJECT_ID_IN_SNAPSHOT) == false) {
18421844
DiffableUtils.jdkMapDiffWithUpdatedKeys(mapDiff, projectRepo -> {
18431845
if (ProjectId.DEFAULT.equals(projectRepo.projectId()) == false) {
18441846
final var message = "Cannot write instance with non-default project id "
18451847
+ projectRepo.projectId()
18461848
+ " to version before "
1847-
+ TransportVersions.PROJECT_ID_IN_SNAPSHOT;
1849+
+ PROJECT_ID_IN_SNAPSHOT;
18481850
assert false : message;
18491851
throw new IllegalArgumentException(message);
18501852
}

‎server/src/main/java/org/elasticsearch/cluster/block/ClusterBlocks.java‎

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

1212
import org.elasticsearch.TransportVersion;
13-
import org.elasticsearch.TransportVersions;
1413
import org.elasticsearch.cluster.Diff;
1514
import org.elasticsearch.cluster.Diffable;
1615
import org.elasticsearch.cluster.SimpleDiffable;
@@ -46,6 +45,7 @@ public class ClusterBlocks implements Diffable<ClusterBlocks> {
4645

4746
public static final ClusterBlocks EMPTY_CLUSTER_BLOCK = new ClusterBlocks(Set.of(), Map.of());
4847

48+
private static final TransportVersion MULTI_PROJECT = TransportVersion.fromName("multi_project");
4949
private static final TransportVersion PROJECT_DELETION_GLOBAL_BLOCK = TransportVersion.fromName("project_deletion_global_block");
5050

5151
private final Set<ClusterBlock> global;
@@ -392,7 +392,7 @@ public String toString() {
392392

393393
@Override
394394
public void writeTo(StreamOutput out) throws IOException {
395-
if (out.getTransportVersion().onOrAfter(TransportVersions.MULTI_PROJECT)) {
395+
if (out.getTransportVersion().supports(MULTI_PROJECT)) {
396396
writeBlockSet(global, out);
397397
out.writeMap(projectBlocksMap, (o, projectId) -> projectId.writeTo(o), (o, projectBlocks) -> projectBlocks.writeTo(out));
398398
} else {
@@ -467,7 +467,7 @@ public ClusterBlocks apply(ClusterBlocks part) {
467467
*/
468468
@Override
469469
public void writeTo(StreamOutput out) throws IOException {
470-
if (out.getTransportVersion().onOrAfter(TransportVersions.MULTI_PROJECT)) {
470+
if (out.getTransportVersion().supports(MULTI_PROJECT)) {
471471
out.writeBoolean(true);
472472
part.writeTo(out);
473473
} else {
@@ -488,7 +488,7 @@ private static void writeBlockSet(Set<ClusterBlock> blocks, StreamOutput out) th
488488
}
489489

490490
public static ClusterBlocks readFrom(StreamInput in) throws IOException {
491-
if (in.getTransportVersion().onOrAfter(TransportVersions.MULTI_PROJECT)) {
491+
if (in.getTransportVersion().supports(MULTI_PROJECT)) {
492492
final Set<ClusterBlock> global = readBlockSet(in);
493493
final Map<ProjectId, ProjectBlocks> projectBlocksMap = in.readImmutableMap(ProjectId::readFrom, ProjectBlocks::readFrom);
494494
if (global.isEmpty()
@@ -520,7 +520,7 @@ private static Set<ClusterBlock> readBlockSet(StreamInput in) throws IOException
520520

521521
public static Diff<ClusterBlocks> readDiffFrom(StreamInput in) throws IOException {
522522
if (in.readBoolean()) {
523-
if (in.getTransportVersion().onOrAfter(TransportVersions.MULTI_PROJECT)) {
523+
if (in.getTransportVersion().supports(MULTI_PROJECT)) {
524524
return new ClusterBlocksDiff(ClusterBlocks.readFrom(in), false);
525525
} else {
526526
return new ClusterBlocksDiff(ClusterBlocks.readFromSingleProjectNode(in), true);

0 commit comments

Comments
 (0)