Skip to content

Bump min Cassandra package #2836

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

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

thompson-tomo
Copy link

@thompson-tomo thompson-tomo commented Jun 11, 2025

Changes

In the 3.1.16 version of Cassandra the upper bound limit on System.Threading.Tasks.Dataflow has been known to cause conflicts on newer TFM so package has been bumped 1 minor to address that

Merge requirement checklist

  • CONTRIBUTING guidelines followed (license requirements, nullable enabled, static analysis, etc.)
  • Unit tests added/updated
  • Appropriate CHANGELOG.md files updated for non-trivial changes
  • Changes in public API reviewed (if applicable)

@thompson-tomo thompson-tomo requested a review from a team as a code owner June 11, 2025 14:02
@github-actions github-actions bot added the infra Infra work - CI/CD, code coverage, linters label Jun 11, 2025
@martincostello
Copy link
Member

martincostello commented Jun 11, 2025

Looked it up and the effective diff in the dependencies of CassandraCSharpDriver is:

- System.Threading.Tasks.Dataflow (>= 4.6.0 && < 5.0.0)
+ System.Threading.Tasks.Dataflow (>= 4.6.0)

CHANGELOG

TL;DR - mostly bug fixes, plus that dependency change and one other new feature.

Copy link
Member

@martincostello martincostello left a comment

Choose a reason for hiding this comment

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

The CHANGELOG needs updating.

@github-actions github-actions bot added the comp:instrumentation.cassandra Things related to OpenTelemetry.Instrumentation.Cassandra label Jun 11, 2025
@thompson-tomo
Copy link
Author

Sorry, wasn't sure what is considered a trivial change. Is done now.

thompson-tomo and others added 3 commits June 13, 2025 20:51
Co-authored-by: Martin Costello <martin@martincostello.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
comp:instrumentation.cassandra Things related to OpenTelemetry.Instrumentation.Cassandra infra Infra work - CI/CD, code coverage, linters
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants