-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Add equals operator to Atlas Search #1606
Conversation
|
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.
Just a couple of comments - possibly no changes needed.
driver-core/src/main/com/mongodb/client/model/search/SearchOperator.java
Show resolved
Hide resolved
driver-core/src/main/com/mongodb/client/model/search/SearchOperator.java
Show resolved
Hide resolved
driver-core/src/main/com/mongodb/client/model/search/SearchOperator.java
Show resolved
Hide resolved
} | ||
|
||
/** | ||
* Returns a {@link SearchOperator} that checks whether a field matches a value you specify. |
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.
The docs from our docs page are confusing. I think we should say, ...that searches for documents where a field matches the specified value.
in all of these
driver-core/src/main/com/mongodb/client/model/search/SearchOperator.java
Show resolved
Hide resolved
# Conflicts: # driver-core/src/main/com/mongodb/client/model/search/SearchConstructibleBsonElement.java # driver-core/src/main/com/mongodb/client/model/search/SearchOperator.java # driver-core/src/test/functional/com/mongodb/client/model/search/AggregatesSearchIntegrationTest.java # driver-core/src/test/unit/com/mongodb/client/model/search/SearchOperatorTest.java # driver-scala/src/main/scala/org/mongodb/scala/model/search/SearchOperator.scala # driver-scala/src/main/scala/org/mongodb/scala/model/search/package.scala
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.
The last reviewed commit is 013310a. I have not fully reviewed the commits before it.
I reviewed the commits before that commit. |
JAVA-5729 --------- Co-authored-by: Maxim Katcharov <maxim.katcharov@mongodb.com>
Ticket
JAVA-5729
Description
There are several Atlas Search query operators that are not implemented with first class support in the Java driver. This PR adds the
equals
operator to Atlas Search.Testing
./gradlew check
atlas-search-test
on EvergreenSearchOperatorTest