Add todos to update version

Signed-off-by: Lakshya Taragi <[email protected]>
ltaragi · Sep 23, 2024 · a706f7d · a706f7d
1 parent fd373da
commit a706f7d
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 0 deletions.
diff --git a/server/src/main/java/org/opensearch/action/admin/cluster/node/stats/NodeStats.java b/server/src/main/java/org/opensearch/action/admin/cluster/node/stats/NodeStats.java
@@ -247,6 +247,7 @@ public NodeStats(StreamInput in) throws IOException {
         } else {
             nodeCacheStats = null;
         }
+        // TODO: change version to V_2_18_0
         if (in.getVersion().onOrAfter(Version.CURRENT)) {
             remoteStoreNodeStats = in.readOptionalWriteable(RemoteStoreNodeStats::new);
         } else {
@@ -541,6 +542,7 @@ public void writeTo(StreamOutput out) throws IOException {
         if (out.getVersion().onOrAfter(Version.V_2_14_0)) {
             out.writeOptionalWriteable(nodeCacheStats);
         }
+        // TODO: change version to V_2_18_0
         if (out.getVersion().onOrAfter(Version.CURRENT)) {
             out.writeOptionalWriteable(remoteStoreNodeStats);
         }

diff --git a/server/src/main/java/org/opensearch/node/remotestore/RemoteStoreNodeStats.java b/server/src/main/java/org/opensearch/node/remotestore/RemoteStoreNodeStats.java
@@ -41,13 +41,15 @@ public long getLastSuccessfulFetchOfPinnedTimestamps() {
     }
 
     public RemoteStoreNodeStats(StreamInput in) throws IOException {
+        // TODO: change version to V_2_18_0
         if (in.getVersion().onOrAfter(Version.CURRENT) && RemoteStoreSettings.isPinnedTimestampsEnabled()) {
             this.lastSuccessfulFetchOfPinnedTimestamps = in.readOptionalLong();
         }
     }
 
     @Override
     public void writeTo(StreamOutput out) throws IOException {
+        // TODO: change version to V_2_18_0
         if (out.getVersion().onOrAfter(Version.CURRENT) && RemoteStoreSettings.isPinnedTimestampsEnabled()) {
             out.writeOptionalLong(this.lastSuccessfulFetchOfPinnedTimestamps);
         }