-
Notifications
You must be signed in to change notification settings - Fork 32
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: Change core.DocumentMapping
to pointer
#1528
fix: Change core.DocumentMapping
to pointer
#1528
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #1528 +/- ##
===========================================
+ Coverage 72.01% 72.12% +0.10%
===========================================
Files 185 185
Lines 18295 18295
===========================================
+ Hits 13175 13195 +20
+ Misses 4072 4058 -14
+ Partials 1048 1042 -6
|
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.
LGTM praise for this find
question: Why did change detector need to be part of this PR? is there a correlation between the pointer fix and change detector?
I could of done a separate PR for it but it was such a small change and this PR wasn't passing the change detector check without it. Two birds one stone. |
## Relevant issue(s) Resolves sourcenetwork#1526 ## Description This PR converts `core.DocumentMapping` in `mapper.Select` and `mapper.Aggregate` to a pointer. It also adds an integration test for the two level deep filter. It also includes a fix of the change detector for when there are no other actions after SchemaUpdate, CreateDoc and UpdateDoc. *Note: The pointer fix was found by @jsimnz.*
## Relevant issue(s) Resolves sourcenetwork#1530 ## Description This PR fixes the previous fix and adds a breaking change document to prevent the broken cycle in the change detector after the fix. Should have been part of the sourcenetwork#1528 although the change detector was passing before merge.
Relevant issue(s)
Resolves #1526
Description
This PR converts
core.DocumentMapping
inmapper.Select
andmapper.Aggregate
to a pointer. It also adds an integration test for the two level deep filter.It also includes a fix of the change detector for when there are no other actions after SchemaUpdate, CreateDoc and UpdateDoc.
Note: The pointer fix was found by @jsimnz.
Tasks
How has this been tested?
make test
Specify the platform(s) on which this was tested: