You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The RediSearch search command returns an instance of the Result class except when the Redis client is a Pipeline because Pipeline returns itself instead of a result when you execute a command. There's code that checks for this in both the SearchCommands and AsyncSearchCommands classes:
I'm not sure if it makes sense to check for both Pipeline types or if the SearchCommands class should check for just Pipeline and the AsyncSearchCommands class should check for just AsyncPipeline. Let me know and I can make a PR. Or feel free to make the changes yourself if that's easier. Thanks!
The text was updated successfully, but these errors were encountered:
The RediSearch search command returns an instance of the
Result
class except when the Redis client is aPipeline
becausePipeline
returns itself instead of a result when you execute a command. There's code that checks for this in both theSearchCommands
andAsyncSearchCommands
classes:redis-py/redis/commands/search/commands.py
Line 414 in 4b0543d
redis-py/redis/commands/search/commands.py
Line 883 in 4b0543d
However, this check doesn't work if the
Pipeline
is from theredis.asyncio.client
module. The following modification should fix the issue:I'm not sure if it makes sense to check for both
Pipeline
types or if theSearchCommands
class should check for justPipeline
and theAsyncSearchCommands
class should check for justAsyncPipeline
. Let me know and I can make a PR. Or feel free to make the changes yourself if that's easier. Thanks!The text was updated successfully, but these errors were encountered: