Expressions final #1073
Merged
Expressions final #1073
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
stIncMale
approved these changes
Jan 27, 2023
katcharov
force-pushed
the
expressions-docs2
branch
from
7afdf1f to
3384440
Compare
katcharov
force-pushed
the
expressions-renames
branch
from
024b85c to
1152643
Compare
Collaborator
Author
|
Manually squashed some commits. |
katcharov
force-pushed
the
expressions
branch
from
820e7f8 to
520ae92
Compare
katcharov
force-pushed
the
expressions-renames
branch
from
3f8eb75 to
594c380
Compare
katcharov
force-pushed
the
expressions
branch
from
520ae92 to
173c3a8
Compare
katcharov
force-pushed
the
expressions-renames
branch
from
594c380 to
581dad2
Compare
katcharov
force-pushed
the
expressions-renames
branch
from
581dad2 to
28c2833
Compare
katcharov
added a commit
that referenced
this pull request
Jan 31, 2023
katcharov
added a commit
that referenced
this pull request
Jan 31, 2023
katcharov
added a commit
that referenced
this pull request
Jan 31, 2023
katcharov
added a commit
that referenced
this pull request
Jan 31, 2023
* Implement boolean expressions (#1025) JAVA-4779 * Implement filter, map, reduce (#1031) JAVA-4781 * Implement eq, ne, gt, gte, lt, lte (#1033) JAVA-4784 * Implement string expressions (#1036) JAVA-4801 * Implement arithmetic expressions (#1037) Implement arithmetic expressions (from top 50, and others) JAVA-4803 * Implement array expressions (#1043) JAVA-4805 * Implement date expressions (#1045) JAVA-4804 * Implement conversion/type expressions (#1050) JAVA-4802 * Implement document expressions (#1052) JAVA-4782 * Replace reduce with individual reductions (#1053) JAVA-4814 * Implement map expressions (#1054) JAVA-4817 * Implement switch expression (#1055) JAVA-4813 * Test expressions in context (#1057) JAVA-4820 * Add javadoc for boolean, date, number, integer, and expression (#1059) JAVA-4799 * Update and add documentation (#1059) * Fix, tests JAVA-4799 * Add `@MqlUnchecked` and a few usage examples (#1059) JAVA-4799 * Add has to document, add tests (#1070) JAVA-4799 * Add javadocs for remaining classes (#1070) JAVA-4799 * 5.2 annotations (#1070) JAVA-4799 * 5.0 annotations (#1070) JAVA-4799 * 4.4 annotations (#1070) JAVA-4799 * 4.2 annotations (#1070) JAVA-4799 * 4.0 annotations (#1070) JAVA-4799 * Update and add documentation, add tests, fix minor issues (#1070) Rename extractBsonValue Fix access modifiers Remove excess comments Update docs Fix: behaviour of get Add notNull to API, add notNullApi test Fix docs/annotations, tests Fix docs, annotations, since Fix docs Revert external Add missing MqlUnchecked Fix missing null checks Checkstyle JAVA-4799 * Rename to Mql (automated) (#1073) JAVA-3879 * Rename methods (automated) (#1073) JAVA-3879 * Update naming, terms, and missing checks and annotations (#1073) JAVA-3879 --------- Co-authored-by: Valentin Kovalenko <valentin.kovalenko@mongodb.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Outstanding issues from JAVA-3879, primarily renames.
Largest renames were Expression->Mql, but this also includes (in separated commits) renames of methods that seemed inconsistent with similar methods, or which were otherwise problematic (for example,
strLenbecamelength,concat(fn)was renamed toconcatArraysto distinguish it from array-arrayconcat, andhaswas renamed tohasFieldfor consistency with the other document accessors, and so on).A dubious convenience method was removed, and missed null-checks and annotations were added (noticed in a prior PR, but too late to change there due to the scope of these renames).
Renames that were performed exclusively using automated tooling are in separate commits marked "automated".