Skip to content

JAVA-6237: Explicit Encryption Case 2 uses contention-10 collection#2003

Open
rozza wants to merge 1 commit into
mongodb:mainfrom
rozza:JAVA-6237
Open

JAVA-6237: Explicit Encryption Case 2 uses contention-10 collection#2003
rozza wants to merge 1 commit into
mongodb:mainfrom
rozza:JAVA-6237

Conversation

@rozza

@rozza rozza commented Jun 18, 2026

Copy link
Copy Markdown
Member

Servers implementing SERVER-91887 require payloads to exactly match the collection's configured contention factor, not just avoid exceeding it.

Updates the specifications submodule to d30d717b

JAVA-6237

Servers implementing SERVER-91887 require payloads to exactly match
the collection's configured contention factor, not just avoid exceeding it.

Updates the specifications submodule to d30d717b (specifications master).
@rozza rozza requested a review from a team as a code owner June 18, 2026 09:02
@rozza rozza requested a review from stIncMale June 18, 2026 09:02

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

[optional]

Let's add

  • A class-level Java documentation comment with a URL to the spec file containing the prose tests.
  • A method-level Java documentation comment with the test name (whatever is needed to find the test in the document) on each test method.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This PR needs to revert the change to driver-sync/src/test/functional/com/mongodb/client/AbstractClientSideEncryptionExplicitEncryptionTest.java done in #2001.

@stIncMale stIncMale Jul 3, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This change updates the submodule to a commit that is older than the the submodule commit in main. Let's merge main into the PR branch. Such a merge is also needed for addressing #2003 (comment).

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.

2 participants