Moti Cohen
Moti Cohen
Hi @sjperkins, Thanks for the PR! I'd suggest - Using `destroy` instead of `close` for better semantic accuracy. This callback doesn't just close a handle it destroys the entire context...
@alonre24, > I reviewed the module API part For RediSearch usecase, we are going to use metadata as a pointer (to an array of document ids) - so it would...
> I was interested in testing the serializing and deserializing of the metadata in cases where the metadata is a pointer (simulating RDB store/load) @alonre24, I didn't implement yet RDB...
> Hey @moticless , > > would it be possible also to clarify the intended usage of `defrag`, `free_effort` and `mem_usage`? Hey @JoanFM, Don't worry about these for now, but...
@oranagra, i can move it to librdb repo and maintain it there if you have doubts.
> @moticless We have a RESP reference [here](https://redis.io/docs/latest/develop/reference/protocol-spec/) with it's source [here](https://github.com/redis/docs/blob/main/content/develop/reference/protocol-spec.md). > > Maybe it would be a better fit to have RDB files documented near that? @collinfunk, according...
> H[P]TTL (this is probably a bug, as these 2 commands should be marked as NONDETERMINISTIC just like [P]TTL) Created PR: https://github.com/redis/redis/pull/13461
Hi @RaphaelDrai, Thanks a lot for putting together the Redis module. You placed it directly inside the main Redis repo. Modules should be maintained independently. Could you please move it...
> Hi @moticless I moved the RedisTABLE module to a separate repository. How can I get review from Redis reviewers? https://github.com/RedisTABLE/RedisTABLE Thanks Because modules are designed to be self-contained, there...
Agreed. I wasn’t aware that OBJ_ENCODING_EMBSTR_SIZE_LIMIT was formally documented. My original intent was to prioritize cache-line locality and keep the evaluation of key+value simple, rather than making accurate evaluation. This...