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.

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

Fix EphemeralDB index update after document deletion #306

Merged

Conversation

bouthilx
Copy link
Member

Why:

When remove documents, the corresponding indexes would not be deleted,
making impossible to insert new documents with these values.

Why:

When remove documents, the corresponding indexes would not be deleted,
making impossible to insert new documents with these values.
@bouthilx bouthilx added bug Indicates an unexpected problem or unintended behavior v0.1.8 labels Oct 18, 2019
@bouthilx bouthilx requested a review from Delaunay October 18, 2019 16:23
@bouthilx
Copy link
Member Author

@codecov-io
Copy link

Codecov Report

Merging #306 into develop will increase coverage by 0.01%.
The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #306      +/-   ##
===========================================
+ Coverage    94.87%   94.88%   +0.01%     
===========================================
  Files           62       62              
  Lines         9873     9899      +26     
  Branches       236      236              
===========================================
+ Hits          9367     9393      +26     
  Misses         482      482              
  Partials        24       24
Impacted Files Coverage Δ
tests/unittests/core/test_ephemeraldb.py 100% <100%> (ø) ⬆️
tests/unittests/core/test_pickleddb.py 97.08% <100%> (+0.18%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3604466...a857148. Read the comment docs.

@bouthilx bouthilx merged commit bca8778 into Epistimio:develop Oct 18, 2019
This was referenced Jun 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Indicates an unexpected problem or unintended behavior v0.1.8
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants