Skip to content

Conversation

@Fudoshiki
Copy link
Contributor

No description provided.

Copilot AI review requested due to automatic review settings November 20, 2025 17:37
Copilot finished reviewing on behalf of Fudoshiki November 20, 2025 17:39
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds support for Erlang/OTP 28's built-in :zstd module while maintaining backward compatibility with the :ezstd library for older OTP versions.

Key Changes

  • Implemented runtime detection that prioritizes OTP 28's native :zstd module over the :ezstd library
  • Centralized compressor availability logic in Mongo.Compressor module
  • Updated documentation to reflect the dual support approach

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
File Description
lib/mongo/compressor.ex Added module detection logic with :zstd priority over :ezstd, refactored all compress/uncompress functions to use the detected module
lib/mongo/url_parser.ex Replaced hardcoded compressor list with dynamic call to Mongo.Compressor.compressors()
lib/mongo/server_description.ex Updated to use centralized compressor list from Mongo.Compressor module, simplified support_compressors/0 function
README.md Updated compression documentation to explain OTP 28+ native support and formatting improvements (list conversions, whitespace)
CHANGELOG.md Added unreleased section documenting the zstd compression enhancement and reformatted existing entries to use consistent markdown list syntax

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@zookzook
Copy link
Owner

Thanks for the PR. This makes sense!

@zookzook zookzook merged commit 79053ac into zookzook:master Nov 22, 2025
2 of 9 checks passed
@zookzook
Copy link
Owner

I will take care of the failing test.

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