Conversation
* oom is 80% hard coded. basic shard log * added error message in FT.INFO * remove redundant additions * treat maxmemry == 0 is invalid * fixed PR comments * fixed a typo * indent * draft * Add warning to resp3 * Add background indexing failure to info metrics and ft.info * deserilize error * format * correct config * add config to config pytest * add initial basic pytest * Raise bg_index_error and temporal solution for % in log * base verbosity test * revert resp3 warning * fix missing comma * add pause on OOM mechanism * Add test for pause on OOM mechanism * New tests * new test * error on OOM index * create new test file * revert text_index_error to master * Add OOM check for ALTER and test * cleanup + format * little cleanup * more cleanup * change % to percent for log display * spellcheck fix * typo fix * fix pytests for index errors field * spellcheck * update index errors dict * fix error change in config, fix pytest for index_errors * revert test config * update IndexError_Deserialize * Leak possible fix * Add terminate bg pool debug command * expose reindex thread pool * expose reindex thread pool * less num_docs, add thread pool terminate * fix pytest * coord into account for ftinfo * fix info for coord * fix serialization process * fix test * fix flakeness * Guy comments round 1 * new help for config * format * change config name * remove Dvir's comment * change error message + format * format, guys comments, changing SET_BG_INDEX_RESUME to be without args, changing debug command syntax * move scanner canceled into memory check blcok * fix query error string * change true/false to macro * adhere to user data * fix config text * fix config * fix config name, add comments on thread pool, change assert * fix pytest * Alon's comments round1 * remove SetIndexErrorMessage * fail of ft.debug search/agg , add assertion instead of if * change test * Fail on hset after OOM, test for cluster * fix assert, fix error message, change loose memory for tests, * fix leakage * fix "missing" error message * support python < 3.9 * fix delete during indexing test, add warning and comment to GIL release * make test more robust --------- Co-authored-by: DvirDukhan <dvir@redis.com> (cherry picked from commit 8b63157)
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## 8.0 #5907 +/- ##
==========================================
+ Coverage 87.25% 87.31% +0.06%
==========================================
Files 206 206
Lines 36523 36635 +112
==========================================
+ Hits 31867 31987 +120
+ Misses 4656 4648 -8 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
alonre24
approved these changes
Apr 10, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
backport PR #5778 to 8.0