Skip to content

[8.0] [MOD-8125] Create inverted index in write operations only #5259

Merged
meiravgri merged 2 commits into8.0from
backport-5229-to-8.0
Nov 27, 2024
Merged

[8.0] [MOD-8125] Create inverted index in write operations only #5259
meiravgri merged 2 commits into8.0from
backport-5229-to-8.0

Conversation

@meiravgri
Copy link
Copy Markdown
Collaborator

backport #5229 to 8.0

cofilct due to debug list format in test_debug_commands

* open vector for read

* don't create vector index  in read mode

* use  OPEN_INDEX_READ or OPEN_INDEX_WRITE in tags

* remove unused openindex function

* create vecsim index to capture info in debug commands

add create_if_missing variable to OpenGeometryIndex

 dont create text index in gc if doesn't exist

* use DONT_CREATE_INDEX in NewVectorIterator

* introduce FT.DEBUG SPEC_INVIDXES_INFO INDEX_NAME

introduce test_index to test lazy intialziation

* fix

* cosmetics

* skip testSpecIndexesInfo in coord

* rename openVectorKeysDict to openVectorIndex

add depth arg to pytest callbacks

add test_lazy_index_creation_info_modules

some tests improvments

* fix field name in test

* pass

(cherry picked from commit 7617563)
@meiravgri meiravgri requested a review from alonre24 November 26, 2024 12:42
@codecov
Copy link
Copy Markdown

codecov bot commented Nov 26, 2024

Codecov Report

Attention: Patch coverage is 98.21429% with 1 line in your changes missing coverage. Please review.

Project coverage is 86.65%. Comparing base (bd0ef0d) to head (9121088).
Report is 1 commits behind head on 8.0.

Files with missing lines Patch % Lines
src/fork_gc.c 75.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##              8.0    #5259      +/-   ##
==========================================
+ Coverage   86.58%   86.65%   +0.06%     
==========================================
  Files         192      192              
  Lines       34407    34420      +13     
==========================================
+ Hits        29793    29825      +32     
+ Misses       4614     4595      -19     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@meiravgri meiravgri enabled auto-merge November 27, 2024 17:07
@meiravgri meiravgri added this pull request to the merge queue Nov 27, 2024
Merged via the queue into 8.0 with commit b40951c Nov 27, 2024
@meiravgri meiravgri deleted the backport-5229-to-8.0 branch November 27, 2024 18:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants