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.

Sign up for GitHub

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

Jump to bottom

[client-common] Added safeguard for compressor #1307

Merged
merged 6 commits into from
Nov 15, 2024
Merged

[client-common] Added safeguard for compressor #1307

merged 6 commits into from
Nov 15, 2024

Commits on Nov 15, 2024

  1. [client-common] Added safeguard for compressor 

    Today, the `compress`/`decompress` can still be invoked
even the compressor is closed already and for zstd based
compressor, it would crash.
This PR add some safeguard and fail fast if the compressor
is already closed.
    @gaojieliu
    gaojieliu committed Nov 15, 2024
    Configuration menu
    Copy the full SHA
    9d948b0 View commit details
    Browse the repository at this point in the history

  2. Fixed integration test failures

    @gaojieliu
    gaojieliu committed Nov 15, 2024
    Configuration menu
    Copy the full SHA
    6b860c8 View commit details
    Browse the repository at this point in the history

  3. Minor tweak

    @gaojieliu
    gaojieliu committed Nov 15, 2024
    Configuration menu
    Copy the full SHA
    be94e25 View commit details
    Browse the repository at this point in the history

  4. Added a unit test

    @gaojieliu
    gaojieliu committed Nov 15, 2024
    Configuration menu
    Copy the full SHA
    8a0ce3a View commit details
    Browse the repository at this point in the history

  5. Fixed minor comment

    @gaojieliu
    gaojieliu committed Nov 15, 2024
    Configuration menu
    Copy the full SHA
    d8064d4 View commit details
    Browse the repository at this point in the history

  6. Skipped locking for NoopCompressor

    @gaojieliu
    gaojieliu committed Nov 15, 2024
    Configuration menu
    Copy the full SHA
    8e8ec26 View commit details
    Browse the repository at this point in the history