-
Notifications
You must be signed in to change notification settings - Fork 508
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.
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
MOD-6534: Missing values indexing - Add support to indexing pipeline #4638
MOD-6534: Missing values indexing - Add support to indexing pipeline #4638
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great 💯
See few comments
src/redisearch_api.c
Outdated
@@ -44,6 +44,7 @@ RefManager* RediSearch_CreateIndex(const char* name, const RSIndexOptions* optio | |||
IndexSpec* spec = NewIndexSpec(name); | |||
StrongRef ref = StrongRef_New(spec, (RefManager_Free)IndexSpec_Free); | |||
IndexSpec_MakeKeyless(spec); | |||
spec->missingFieldDict = dictCreate(&dictTypeHeapStrings, NULL); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add to IndexSpec_MakeKeyless()
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
src/fork_gc.c
Outdated
RedisSearchCtx_UnlockSpec(sctx); | ||
StrongRef_Release(spec_ref); | ||
} | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Free fieldName
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK. Done
5aa4ede
into
feature_missing_values_indexing
Description
A clear and concise description of what the PR is solving, including:
Which issues this PR fixes
Main objects this PR modified
Mark if applicable