redisai-py icon indicating copy to clipboard operation
redisai-py copied to clipboard

modelset chunk size should be configurable

Open filipecosta90 opened this issue 3 years ago • 2 comments

Given https://github.com/redis/redis/blob/unstable/redis.conf#L1854 there might be cases in which we have a Redis instance with a max string size of 1,2,10mb.... The hotfix that added modelset chunking to the client ( https://github.com/RedisAI/redisai-py/pull/35/files ) enforces a chunk size of 512mb. We should address this ( or at least be aware of the limitations/issues that can arise from the current config )

filipecosta90 avatar Apr 14 '21 15:04 filipecosta90

@filipecosta90 can we get this value from CONFIG GET command?

DvirDukhan avatar Apr 17 '21 21:04 DvirDukhan

@filipecosta90 can we get this value from CONFIG GET command?

yes :

127.0.0.1:6379> config get proto-max-bulk-len
1) "proto-max-bulk-len"
2) "536870912"

filipecosta90 avatar Apr 18 '21 08:04 filipecosta90