From bd12e00b5305570bb6442026518de0989dcd2981 Mon Sep 17 00:00:00 2001
From: Luca Cavanna <javanna@users.noreply.github.com>
Date: Tue, 18 Dec 2018 10:33:12 +0100
Subject: [PATCH] Update versions in SearchSortValues transport serialization

Now that #36617 is backported to 6.x, the version in the transport serialization conditionals for the search sort values can be updated to 6.6.0
---
 .../org/elasticsearch/search/SearchSortValues.java     |  4 ++--
 .../elasticsearch/search/SearchSortValuesTests.java    | 10 ++++------
 2 files changed, 6 insertions(+), 8 deletions(-)

diff --git a/server/src/main/java/org/elasticsearch/search/SearchSortValues.java b/server/src/main/java/org/elasticsearch/search/SearchSortValues.java
index 13829665444..bb4a0f63447 100644
--- a/server/src/main/java/org/elasticsearch/search/SearchSortValues.java
+++ b/server/src/main/java/org/elasticsearch/search/SearchSortValues.java
@@ -67,7 +67,7 @@ public class SearchSortValues implements ToXContentFragment, Writeable {
 
     SearchSortValues(StreamInput in) throws IOException {
         this.formattedSortValues = in.readArray(Lucene::readSortValue, Object[]::new);
-        if (in.getVersion().onOrAfter(Version.V_7_0_0)) {
+        if (in.getVersion().onOrAfter(Version.V_6_6_0)) {
             this.rawSortValues = in.readArray(Lucene::readSortValue, Object[]::new);
         } else {
             this.rawSortValues = EMPTY_ARRAY;
@@ -77,7 +77,7 @@ public class SearchSortValues implements ToXContentFragment, Writeable {
     @Override
     public void writeTo(StreamOutput out) throws IOException {
         out.writeArray(Lucene::writeSortValue, this.formattedSortValues);
-        if (out.getVersion().onOrAfter(Version.V_7_0_0)) {
+        if (out.getVersion().onOrAfter(Version.V_6_6_0)) {
             out.writeArray(Lucene::writeSortValue, this.rawSortValues);
         }
     }
diff --git a/server/src/test/java/org/elasticsearch/search/SearchSortValuesTests.java b/server/src/test/java/org/elasticsearch/search/SearchSortValuesTests.java
index 797b5dd888e..bb89d583957 100644
--- a/server/src/test/java/org/elasticsearch/search/SearchSortValuesTests.java
+++ b/server/src/test/java/org/elasticsearch/search/SearchSortValuesTests.java
@@ -136,19 +136,17 @@ public class SearchSortValuesTests extends AbstractSerializingTestCase<SearchSor
         return new SearchSortValues(values);
     }
 
-    //TODO rename and update version after backport
-    public void testSerializationPre70() throws IOException {
-        Version version = VersionUtils.randomVersionBetween(random(), Version.V_6_0_0, VersionUtils.getPreviousVersion(Version.V_7_0_0));
+    public void testSerializationPre6_6_0() throws IOException {
+        Version version = VersionUtils.randomVersionBetween(random(), Version.V_6_0_0, VersionUtils.getPreviousVersion(Version.V_6_6_0));
         SearchSortValues original = createTestInstance();
         SearchSortValues deserialized = copyInstance(original, version);
         assertArrayEquals(original.getFormattedSortValues(), deserialized.getFormattedSortValues());
         assertEquals(0, deserialized.getRawSortValues().length);
     }
 
-    //TODO rename method and adapt versions after backport
-    public void testReadFromPre70() throws IOException {
+    public void testReadFromPre6_6_0() throws IOException {
         try (StreamInput in = StreamInput.wrap(Base64.getDecoder().decode("AwIAAAABAQEyBUAIAAAAAAAAAAAAAAAA"))) {
-            in.setVersion(VersionUtils.randomVersionBetween(random(), Version.V_6_0_0, VersionUtils.getPreviousVersion(Version.V_7_0_0)));
+            in.setVersion(VersionUtils.randomVersionBetween(random(), Version.V_6_0_0, VersionUtils.getPreviousVersion(Version.V_6_6_0)));
             SearchSortValues deserialized = new SearchSortValues(in);
             SearchSortValues expected = new SearchSortValues(new Object[]{1, "2", 3d});
             assertEquals(expected, deserialized);