Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

MAXDOCTABLESIZE not registered into redis-server log #1161

Closed
lumina7 opened this issue Apr 3, 2020 · 4 comments · Fixed by #1165
Closed

MAXDOCTABLESIZE not registered into redis-server log #1161

lumina7 opened this issue Apr 3, 2020 · 4 comments · Fixed by #1165
Assignees
@ashtul
Milestone
1.6.12

Comments

@lumina7
Copy link

lumina7 commented Apr 3, 2020

redis-server --loadmodule ./redisearch.so MAXDOCTABLESIZE 2000000

And when the server start...

cursor read size: 1000, cursor max idle (ms): 300000, max doctable size: 1000000, search pool size: 20, index pool size: 8,

Is MAXDOCTABLESIZE properly registered?
@mnunberg
Copy link
Contributor

mnunberg commented Apr 3, 2020

what version of search are you using?

@lumina7
Copy link
Author

lumina7 commented Apr 3, 2020

RediSearch version 99.99.99 (Git=v1.6.6-45-gc13f342f)

@mnunberg
Copy link
Contributor

mnunberg commented Apr 3, 2020

You're right, it's not actually setting it.

@mnunberg
Copy link
Contributor

mnunberg commented Apr 3, 2020

@ashtul It's simply not setting it in the function, though it's validating them. I'm wondering if there aren't other options that we're missing

@ashtul ashtul mentioned this issue Apr 5, 2020
Merged
@emmanuelkeller emmanuelkeller added this to the 1.6.12 milestone Dec 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ashtul ashtul

Labels
None yet
Projects
None yet
Milestone
1.6.12
Development

Successfully merging a pull request may close this issue.

set maxDocTableSize with new arg RediSearch/RediSearch
4 participants
@mnunberg @lumina7 @ashtul @emmanuelkeller