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

Add equals operator to Atlas Search #1606

Merged
merged 18 commits into from
Jan 28, 2025
Merged

Conversation

joykim1005
Copy link
Contributor

@joykim1005 joykim1005 commented Jan 14, 2025

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

  • ran ./gradlew check
  • ran atlas-search-test on Evergreen
  • ran SearchOperatorTest
  • ran all Evergreen patch

@joykim1005 joykim1005 changed the title Java 5729 Add equals operator Jan 14, 2025
@joykim1005 joykim1005 self-assigned this Jan 14, 2025
@joykim1005 joykim1005 changed the title Add equals operator Add equals operator to Atlas Search Jan 14, 2025
@joykim1005
Copy link
Contributor Author

joykim1005 commented Jan 15, 2025

com.mongodb.client.ClientSideEncryptionE…estExternal_withExternalKeyVault__false is failing and is unlikely caused by this PR and is happening in others as well.

@joykim1005 joykim1005 marked this pull request as ready for review January 15, 2025 19:29
@joykim1005 joykim1005 requested a review from katcharov January 15, 2025 19:29
Copy link
Contributor

@katcharov katcharov left a 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.

}

/**
* Returns a {@link SearchOperator} that checks whether a field matches a value you specify.
Copy link
Contributor

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

@stIncMale stIncMale mentioned this pull request Jan 16, 2025
4 tasks
# 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
@katcharov katcharov requested a review from stIncMale January 24, 2025 20:01
Copy link
Member

@stIncMale stIncMale left a 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.

@katcharov katcharov merged commit 0e25654 into mongodb:main Jan 28, 2025
2 checks passed
@katcharov
Copy link
Contributor

I reviewed the commits before that commit.

rozza pushed a commit that referenced this pull request Feb 6, 2025
JAVA-5729

---------

Co-authored-by: Maxim Katcharov <maxim.katcharov@mongodb.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants