Java : Redis Lettuce Mode : OR Conjunction Condition is not wrapping self and chained condition within parenthesis - Stack Overf

admin管理员组

文章数量:1345016

Sample condition

import com.redis.lettucemod.search.SearchOptions;
import com.redis.lettucemod.search.SearchResults;
import com.redis.lettucemod.search.Limit;
import com.redis.lettucemod.search.Document;
import com.redis.query.Query;
import com.redis.search.query.filter.Condition;
...

      Condition statusOrOnHandCases = Query.tag("status").in("A")
              .or(Query.numeric("onHandCases").ge(0));

      System.out.println("statusOrOnHandCases = "+  statusOrOnHandCases.getQuery());
....

Generated Query String

statusOrOnHandCases = @status:{A}|@onHandCases:[0 inf]

Failure stacktrace

io.lettuce.core.RedisCommandExecutionException: Syntax error at offset 15 near onHandCases
    at io.lettuce.core.internal.ExceptionFactory.createExecutionException(ExceptionFactory.java:147)
    at io.lettuce.core.internal.Exceptions.bubble(Exceptions.java:72)
    at io.lettuce.core.internal.Futures.awaitOrCancel(Futures.java:250)
    at io.lettuce.core.FutureSyncInvocationHandler.handleInvocation(FutureSyncInvocationHandler.java:75)
    at io.lettuce.core.internal.AbstractInvocationHandler.invoke(AbstractInvocationHandler.java:80)
    at jdk.proxy2/jdk.proxy2.$Proxy2.ftSearch(Unknown Source)
    at com.my-app.RedisQueryPoc.main(RedisQueryPoc.java:86)
Caused by: io.lettuce.core.RedisCommandExecutionException: Syntax error at offset 158 near onHandCases
    at io.lettuce.core.internal.ExceptionFactory.createExecutionException(ExceptionFactory.java:147)
    at io.lettuce.core.internal.ExceptionFactory.createExecutionException(ExceptionFactory.java:116)
    at io.lettuce.core.protocol.AsyncCommandpleteResult(AsyncCommand.java:120)
    at io.lettuce.core.protocol.AsyncCommandplete(AsyncCommand.java:111)
    at io.lettuce.core.protocol.CommandHandlerplete(CommandHandler.java:745)
    at io.lettuce.core.protocol.CommandHandler.decode(CommandHandler.java:680)
    at io.lettuce.core.protocol.CommandHandler.channelRead(CommandHandler.java:597)

Expected Query String

1. ((@status:{A}) | (@onHandCases:[0 +inf])) --- with wrapping parenthesis at inner and outer OR
2. (@status:{A}) | (@onHandCases:[0 +inf]) --- with wrapping parenthesis at inner condition

• Note - 1's Outer because this will be further connected to series of and conditions later.

• Reference - Section Mapping common SQL predicates to Redis Query Engine in /

SQL : WHERE x='foo' OR y='bar' REDIS : (@x:foo)\|(@y:bar)

Self Assessments and Validation

• In unit tests cases I could only find multi-valued or usage - .java#L108-L128

• Redis Insights UI : I have verified that OR conditions works only with full wrapping parenthesis otherwise it fails with syntax error message Syntax error at offset 15 near onHandCases

Request / Ask

1. Let me know if I am not using the write constructs to chain or conditions.
2. If the construct seems appropriate, then it looks like a potential bug in the OR condition implementation, kindly address and fix. Also this seems like a pretty basic implementation expectation for or condition not sure why this is missed.

Version Details

lettucemod version : 4.2.1 lettucemod-query version : 4.2.1

本文标签：