Skip to content

CDRIVER-5636 make trimFactor and sparsity optional #1677

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

Merged
merged 7 commits into from
Jul 25, 2024

Conversation

kevinAlbs
Copy link
Collaborator

Summary

  • Document sparsity and trimFactor as optional.
  • Add tests from DRIVERS-2927.
  • Always pass sparsity and trimFactor to libmongocrypt if set.

Evergreen patch: https://spruce.mongodb.com/version/669eb9a3c96e7f000700497c

Background & Motivation

MONGOCRYPT-698 changes libmongocrypt to make trimFactor and sparsity optional. If not set, they default to values matching the server.

This PR includes tests proposed in DRIVERS-2927.

Prior to this PR, trimFactor and sparsity were only passed to libmongocrypt if set to a non-zero value. However, a trimFactor of 0 is a accepted value by libmongocrypt. sparsity of 0 results in an error in libmongocrypt.

@kevinAlbs kevinAlbs marked this pull request as ready for review July 24, 2024 12:33
@kevinAlbs kevinAlbs requested a review from eramongodb July 24, 2024 12:33
@kevinAlbs kevinAlbs requested a review from eramongodb July 25, 2024 18:15
@kevinAlbs kevinAlbs merged commit ae91cf1 into mongodb:master Jul 25, 2024
41 of 45 checks passed
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