Skip to content

Conversation

@jansimonb
Copy link
Contributor

@jansimonb jansimonb commented Dec 9, 2025

Proposed Changes

Adds functionality to disable gRPC response compression for Flight queries in the InfluxDB3 C# client.

  • Adds DisableGrpcCompression property to QueryOptions class
  • Adds INFLUX_DISABLE_GRPC_COMPRESSION environment variable support
  • Adds disableGrpcCompression connection string parameter support
  • Logs a warning if both DisableGrpcCompression and CompressionProviders are set (DisableGrpcCompression takes priority)

Testing

  • Unit tests
  • Integration tests verify grpc-accept-encoding request and grpc-encoding response headers
  • NOTE: InfluxDB 3 Core may not compress responses even when client advertises gzip support. Per gRPC spec, servers may choose not to compress regardless of client settings. InfluxDB Cloud typically compresses, but Core may not. For that reason it is recommended to run the integration test also against InfluxDB Cloud.

Checklist

  • CHANGELOG.md updated
  • Rebased/mergeable
  • A test has been added if appropriate
  • Tests pass
  • Commit messages are conventional
  • Sign CLA (if not already signed)

@jansimonb jansimonb marked this pull request as ready for review December 9, 2025 16:40
Copy link
Member

@bednar bednar left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

@jansimonb jansimonb merged commit d2a782f into main Dec 11, 2025
15 checks passed
@jansimonb jansimonb deleted the feat/disable-grpc-compression branch December 11, 2025 20:13
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.

3 participants