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

Merge next-major #6073

Merged
merged 170 commits into from
Feb 23, 2024
Merged

Merge next-major #6073

merged 170 commits into from
Feb 23, 2024

Conversation

jedelbo
Copy link
Contributor

@jedelbo jedelbo commented Nov 30, 2022

Base for the next major release

@cla-bot cla-bot bot added the cla: yes label Nov 30, 2022
jedelbo and others added 6 commits December 13, 2022 11:23
Optimize storage of Decimal128 properties so that the individual values will take up 0 bits (if all nulls or all zero), 32 bits, 64 bits or 128 bits depending on what is needed.
* temporary disable failing c api decimal test
* Revert "update next major to core 13.4.1 (#6310)"

This reverts commit 59764a2.

* appease format checks
ironage and others added 12 commits February 17, 2023 10:23
update next major to core v13.4.2
"Feature/bugfix release"
Introduce a new class - ColletionParent - which a collection will refer to as its owner. This class can be
specialized as an Obj if the nesting level is 0 or a CollectionList if the collection is nested.
Basic support for nested collections
Make it clear that when an Obj is the owner, then the index must be a ColKey
Co-authored-by: Jørgen Edelbo <jorgen.edelbo@mongodb.com>
Co-authored-by: Jørgen Edelbo <jorgen.edelbo@mongodb.com>
jedelbo and others added 8 commits April 18, 2023 14:27
* Handle nullifying links in nested collections
* Clear backlinks related to nested collections
* dump to json support info about nested collections for schema

* reuse logic for printing nested collections

* main logic for expanding nested collections to json, requires to be polished

* more testing for nested containers

* complete algo for printing nested collections in json format

* add testing json files to project

* generate json files option set to false

* run whole test suite

* test nested collections with links

* format checks

* Move out_mixed_json... functionality to Mixed class

* Remove not needed template parameter from CollectionBaseImpl

* Delegate to_json to collections

* remove commented code

* fix audit conflicts

---------

Co-authored-by: Jørgen Edelbo <jorgen.edelbo@mongodb.com>
* Cleanup naming and consolidate update strategy
* Allow a Mixed containing a ref to be stores in ArrayMixed
jedelbo and others added 24 commits January 12, 2024 15:31
This is done by defining the interface explicitly and in a way that
makes it possible to easily change the underlying implementation.
"Feature/BugFix release"
This will make sense both for strings, binaries and nested collections.
Don't leak implementation of BsonDocument and BsonArray to the users.
This will allow us to use argument substitution in more places. In
particuler if TypeOfValue is expected.

The visit function in ConstantNode is split up i 2 steps. First the
value is extracted into a Mixed - that being directly from the query
string or from the arguments. Then the vaule is adapted to what is
needed based on the 'hint' parameter.
Can be used as a reference implementation in tests.
"Feature/bugfix release"
@jedelbo jedelbo marked this pull request as ready for review February 23, 2024 10:18
@jedelbo jedelbo requested a review from nicola-cab February 23, 2024 10:19
Copy link
Member

@nicola-cab nicola-cab left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

May CI be with us!

@jedelbo jedelbo merged commit 7b432a9 into master Feb 23, 2024
32 of 35 checks passed
@jedelbo jedelbo deleted the next-major branch February 23, 2024 11:45
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 24, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants