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

projections: Delete tree nodes recursively #178

Merged

Conversation

the-mikedavis
Copy link
Member

@the-mikedavis the-mikedavis commented Jan 10, 2023

A deletion to a tree node with children also recursively deletes all of the children tree nodes. Previously, deleted children tree nodes were not deleted from projections. This change looks for all children tree nodes of a deleted tree node and issues projection trigger effects for each deleted tree node.

@the-mikedavis the-mikedavis added the bug Something isn't working label Jan 10, 2023
@the-mikedavis the-mikedavis self-assigned this Jan 10, 2023
@codecov
Copy link

codecov bot commented Jan 10, 2023

Codecov Report

Base: 90.86% // Head: 90.82% // Decreases project coverage by -0.04% ⚠️

Coverage data is based on head (6d6d9f2) compared to base (66eb37a).
Patch coverage: 91.30% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #178      +/-   ##
==========================================
- Coverage   90.86%   90.82%   -0.04%     
==========================================
  Files          19       19              
  Lines        3427     3446      +19     
==========================================
+ Hits         3114     3130      +16     
- Misses        313      316       +3     
Flag Coverage Δ
erlang-24 89.46% <91.30%> (-0.12%) ⬇️
erlang-25 89.75% <91.30%> (-0.04%) ⬇️
os-ubuntu-latest 90.71% <91.30%> (-0.04%) ⬇️
os-windows-latest 89.75% <91.30%> (+0.05%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/khepri_machine.erl 94.41% <91.30%> (-0.26%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

src/khepri_machine.erl Outdated Show resolved Hide resolved
@the-mikedavis the-mikedavis changed the title khepri_machine: Track recursive deletion changes projections: Delete tree nodes recursively Jan 11, 2023
@the-mikedavis the-mikedavis marked this pull request as draft January 11, 2023 19:34
@the-mikedavis the-mikedavis marked this pull request as ready for review January 17, 2023 17:34
@the-mikedavis the-mikedavis marked this pull request as draft January 17, 2023 17:38
@the-mikedavis the-mikedavis marked this pull request as ready for review February 7, 2023 22:00
@dumbbell dumbbell merged commit ae2723f into rabbitmq:main Feb 14, 2023
@the-mikedavis the-mikedavis deleted the md-recursive-delete-projection branch February 14, 2023 13:55
@dumbbell dumbbell added this to the v0.7.0 milestone Feb 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants