Handle dropIndex behavior change by rozza · Pull Request #1785 · mongodb/mongo-java-driver

Skip to content

Navigation Menu

Sign in

Appearance settings

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up

Appearance settings

Conversation

@rozza

Copy link

Member

@rozza rozza commented

Aug 12, 2025

@rozza rozza marked this pull request as ready for review

August 13, 2025 07:28

@rozza rozza requested a review from a team as a code owner

August 13, 2025 07:28

@rozza rozza requested review from nhachicha and removed request for a team

August 13, 2025 07:28
def.skipJira("https://jira.mongodb.org/browse/JAVA-5491")
.testContains("client-side-operations-timeout", "dropIndex")
.when(() -> !serverVersionLessThan(8, 3))
.test("client-side-operations-timeout", "operations ignore deprecated timeout options if timeoutMS is set",
Copy link

Collaborator

Choose a reason for hiding this comment

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

Is there an upcoming PR that updates these tests? It seems the only change is that the server returns ok:1 since 8.3. It's unfortunate to disable all these checks?

Copy link

Member Author

Choose a reason for hiding this comment

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

Yeah theres a drivers ticket related to this to fix / change the tests themselves.

nhachicha reacted with thumbs up emoji

@rozza rozza merged commit 7fa05cd into mongodb:main

Aug 13, 2025

52 of 55 checks passed

@rozza rozza deleted the JAVA-5491 branch

August 13, 2025 12:54

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nhachicha nhachicha nhachicha approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@rozza @nhachicha