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

Date decoding strategy, benchmarks and allow changing default BSON settings #83

Merged
merged 6 commits into from
Apr 21, 2024

Conversation

Joannis
Copy link
Member

@Joannis Joannis commented Apr 20, 2024

Supercedes #82

You can now set BSONDecoderSettings.default = .fastPath for faster BSONDecoder - at the cost of flexibility.
BSONDecoderSettings should only be set at the start of your app boot. Do not change it later, it's not thread safe.

@@ -0,0 +1,13 @@
import Benchmark

let benchmarks = {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Totally forgot about Benchmark!

Copy link
Collaborator

@Andrewangeta Andrewangeta left a comment

Choose a reason for hiding this comment

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

Nothing obvious stands out to me. Looks good :shipit:

@Joannis Joannis merged commit 57da7f3 into main Apr 21, 2024
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.

None yet

3 participants