-
Notifications
You must be signed in to change notification settings - Fork 1
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
support all comparison and aggregation functions on extended json fields #99
support all comparison and aggregation functions on extended json fields #99
Conversation
CHANGELOG.md
Outdated
@@ -4,6 +4,14 @@ This changelog documents the changes between release versions. | |||
|
|||
## [Unreleased] | |||
|
|||
### Added | |||
|
|||
- Extended JSON fields no support all comparison and aggregation functions ([#99](https://github.com/hasura/ndc-mongodb/pull/99)) |
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.
- Extended JSON fields no support all comparison and aggregation functions ([#99](https://github.com/hasura/ndc-mongodb/pull/99)) | |
- Extended JSON fields now support all comparison and aggregation functions ([#99](https://github.com/hasura/ndc-mongodb/pull/99)) |
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.
Looks good, apart from the typo in the changelog. We should also allow order_by
on extended json fields, but that could be in a separate PR.
Good point @dmoverton. This PR does add order_by functions, but I didn't put in a test for that. I just added one. |
Since Extended JSON fields may contain any data users may reasonably want access to all comparison and aggregation operations. MongoDB is designed to accommodate functions that don't make sense on all values they are applied to. I included a native query that produces a collection of mixed data types, and some integration tests on that data.
I also updated the dev shell to pull the cli from the nix flake that @TheInnerLight helpfully set up.