Remove RediSearch DIALECT 5 support (#3886)

This is partial revert of #3831. Some changes are still available without dialect 5. For example: * Avoid repeating for numeral equality

Author: sazzad16

src/test/java/redis/clients/jedis/modules/search/SearchTest.java

@@ -406,7 +406,7 @@ public void testQueryParams() {
  Query query = new Query("@numval:[$min $max]").addParam("min", 1).addParam("max", 2).dialect(2);
  assertEquals(2, client.ftSearch(index, query).getTotalResults());
 
- query = new Query("@numval:[$eq]").addParam("eq", 2).dialect(5);
+ query = new Query("@numval:[$eq]").addParam("eq", 2).dialect(4);
  assertEquals(1, client.ftSearch(index, query).getTotalResults());
  }
 
@@ -533,7 +533,7 @@ public void testJsonWithAlias() {
  assertEquals(1, res.getTotalResults());
  assertEquals("king:1", res.getDocuments().get(0).getId());
 
- res = client.ftSearch(index, new Query("@num:[42]").dialect(5));
+ res = client.ftSearch(index, new Query("@num:[42]").dialect(4));
  assertEquals(1, res.getTotalResults());
  assertEquals("king:1", res.getDocuments().get(0).getId());
  }
@@ -777,7 +777,6 @@ public void getTagField() {
  assertEquals(1, client.ftSearch(index, new Query("@category:{yellow}")).getTotalResults());
  assertEquals(0, client.ftSearch(index, new Query("@category:{purple}")).getTotalResults());
  assertEquals(1, client.ftSearch(index, new Query("@category:{orange\\;purple}")).getTotalResults());
- assertEquals(1, client.ftSearch(index, new Query("@category:{orange;purple}").dialect(5)).getTotalResults());
  assertEquals(4, client.ftSearch(index, new Query("hello")).getTotalResults());
 
  assertEquals(new HashSet<>(Arrays.asList("red", "blue", "green", "yellow", "orange;purple")),
@@ -819,7 +818,6 @@ public void testGetTagFieldWithNonDefaultSeparator() {
  assertEquals(1, client.ftSearch(index, new Query("hello @category:{yellow}")).getTotalResults());
  assertEquals(0, client.ftSearch(index, new Query("@category:{purple}")).getTotalResults());
  assertEquals(1, client.ftSearch(index, new Query("@category:{orange\\,purple}")).getTotalResults());
- assertEquals(1, client.ftSearch(index, new Query("@category:{orange,purple}").dialect(5)).getTotalResults());
  assertEquals(4, client.ftSearch(index, new Query("hello")).getTotalResults());
 
  assertEquals(new HashSet<>(Arrays.asList("red", "blue", "green", "yellow", "orange,purple")),

src/test/java/redis/clients/jedis/modules/search/SearchWithParamsTest.java

@@ -602,7 +602,7 @@ public void testQueryParams() {
  .dialect(2)).getTotalResults());
 
  assertEquals(1, client.ftSearch(index, "@numval:[$eq]",
- FTSearchParams.searchParams().addParam("eq", 2).dialect(5)).getTotalResults());
+ FTSearchParams.searchParams().addParam("eq", 2).dialect(4)).getTotalResults());
  }
 
  @Test
@@ -665,7 +665,7 @@ public void testJsonWithAlias() {
  assertEquals(1, res.getTotalResults());
  assertEquals("king:1", res.getDocuments().get(0).getId());
 
- res = client.ftSearch(index, "@num:[42]", FTSearchParams.searchParams().dialect(5));
+ res = client.ftSearch(index, "@num:[42]", FTSearchParams.searchParams().dialect(4));
  assertEquals(1, res.getTotalResults());
  assertEquals("king:1", res.getDocuments().get(0).getId());
  }
@@ -871,8 +871,6 @@ public void getTagField() {
  assertEquals(1, client.ftSearch(index, "@category:{yellow}").getTotalResults());
  assertEquals(0, client.ftSearch(index, "@category:{purple}").getTotalResults());
  assertEquals(1, client.ftSearch(index, "@category:{orange\\;purple}").getTotalResults());
- assertEquals(1, client.ftSearch(index, "@category:{orange;purple}",
- FTSearchParams.searchParams().dialect(5)).getTotalResults());
  assertEquals(4, client.ftSearch(index, "hello").getTotalResults());
 
  assertEquals(new HashSet<>(Arrays.asList("red", "blue", "green", "yellow", "orange;purple")),
@@ -912,8 +910,6 @@ public void testGetTagFieldWithNonDefaultSeparator() {
  assertEquals(1, client.ftSearch(index, "hello @category:{yellow}").getTotalResults());
  assertEquals(0, client.ftSearch(index, "@category:{purple}").getTotalResults());
  assertEquals(1, client.ftSearch(index, "@category:{orange\\,purple}").getTotalResults());
- assertEquals(1, client.ftSearch(index, "@category:{orange,purple}",
- FTSearchParams.searchParams().dialect(5)).getTotalResults());
  assertEquals(4, client.ftSearch(index, "hello").getTotalResults());
 
  assertEquals(new HashSet<>(Arrays.asList("red", "blue", "green", "yellow", "orange,purple")),