Skip to content

Commit c09dae2

Browse files
authored
Make default client side search dialect to 2 (#4060)
* Make default client side search dialect to 2 * Fix tests
1 parent dca716e commit c09dae2

File tree

4 files changed

+1
-10
lines changed

4 files changed

+1
-10
lines changed

src/main/java/redis/clients/jedis/CommandObjects.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ protected RedisProtocol getProtocol() {
5353
private JedisBroadcastAndRoundRobinConfig broadcastAndRoundRobinConfig = null;
5454
private Lock mapperLock = new ReentrantLock(true);
5555
private volatile JsonObjectMapper jsonObjectMapper;
56-
private final AtomicInteger searchDialect = new AtomicInteger(0);
56+
private final AtomicInteger searchDialect = new AtomicInteger(2); // DEFAULT_SEARCH_DIALECT = 2;
5757

5858
@Experimental
5959
void setKeyArgumentPreProcessor(CommandKeyArgumentPreProcessor keyPreProcessor) {

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

-3
Original file line numberDiff line numberDiff line change
@@ -122,9 +122,6 @@ public void createWithFieldNames() {
122122
SearchResult noFilters = client.ftSearch(index, new Query());
123123
assertEquals(5, noFilters.getTotalResults());
124124

125-
SearchResult asOriginal = client.ftSearch(index, new Query("@\\$\\.first:Jo*"));
126-
assertEquals(0, asOriginal.getTotalResults());
127-
128125
SearchResult asAttribute = client.ftSearch(index, new Query("@first:Jo*"));
129126
assertEquals(2, asAttribute.getTotalResults());
130127

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

-3
Original file line numberDiff line numberDiff line change
@@ -139,9 +139,6 @@ public void createWithFieldNames() throws Exception {
139139
SearchResult noFilters = client.ftSearch(index, new Query());
140140
assertEquals(5, noFilters.getTotalResults());
141141

142-
SearchResult asOriginal = client.ftSearch(index, new Query("@first:Jo*"));
143-
assertEquals(0, asOriginal.getTotalResults());
144-
145142
SearchResult asAttribute = client.ftSearch(index, new Query("@given:Jo*"));
146143
assertEquals(2, asAttribute.getTotalResults());
147144

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

-3
Original file line numberDiff line numberDiff line change
@@ -147,9 +147,6 @@ public void createWithFieldNames() {
147147
SearchResult noFilters = client.ftSearch(index);
148148
assertEquals(5, noFilters.getTotalResults());
149149

150-
SearchResult asFirst = client.ftSearch(index, "@first:Jo*");
151-
assertEquals(0, asFirst.getTotalResults());
152-
153150
SearchResult asGiven = client.ftSearch(index, "@given:Jo*");
154151
assertEquals(2, asGiven.getTotalResults());
155152

0 commit comments

Comments
 (0)