Skip to content

Conversation

connorsmacd
Copy link
Contributor

@connorsmacd connorsmacd commented Sep 29, 2025

Server 8.2 supports mixing QE and non-CSFLE JSON schema validators (SERVER-100260). This PR updates the $lookup prose test accordingly.

Case 8: db.csfle joins db.qe

MONGOCRYPT-793 changes libmongocrypt's behavior to no longer error when QE and CSFLE schemas are mixed. While it is still an error if the CSFLE schema contains encrypt validators, the error now comes from the query analysis component with a different message. The test case now specifies different expected error substrings depending on the version of the query analysis component.

Case 10: db.qe joins db.non_csfle_schema (new)

This tests that a $lookup with a both QE and non-CSFLE JSON schema validators completes successfully.


C Driver Reference Implementations:


Please complete the following before merging:

  • Is the relevant DRIVERS ticket in the PR title?
  • Update changelog.
  • Test changes in at least one language driver.
  • Test these changes against all server versions and topologies (including standalone, replica set, and sharded
    clusters).

@connorsmacd connorsmacd marked this pull request as ready for review October 1, 2025 15:00
@connorsmacd connorsmacd requested a review from a team as a code owner October 1, 2025 15:00
@connorsmacd connorsmacd requested review from katcharov and kevinAlbs and removed request for a team October 1, 2025 15:00
Copy link
Contributor

@kevinAlbs kevinAlbs left a comment

Choose a reason for hiding this comment

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

LGTM with minor comments.

@kevinAlbs kevinAlbs requested review from mdb-ad and removed request for katcharov October 3, 2025 17:56
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