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

Move Kafka-related functionality to stracer-kafka module [CE2] #282

Merged
merged 1 commit into from
Jul 7, 2023

Conversation

Z1kkurat
Copy link
Contributor

@Z1kkurat Z1kkurat commented Jul 7, 2023

This is a back-port of #281

This PR moves all Kafka-related functionality into a separate stracer-kafka module. Since it's a breaking change (classes are removed from the main library and it's necessary to bring in a new one), the major version is bumped to 5.0.0.

Motivation: bringing stracer into the classpath automatically means bringing some version of skafka. Sometimes it's simply unnecessary (e.g. when data classes are received via HTTP), and sometimes it can be very inconvenient, e.g. when this library and some other pull different major versions of skafkainto the classpath, but ReportSpanRecord isn't used in the application, however, sbt screams about major version eviction.
Moreover, ReportSpanRecord is additional functionality, not directly related to the data classes that comprise the core of the library.

@Z1kkurat Z1kkurat merged commit 6e8606c into series/2.x.x Jul 7, 2023
@Z1kkurat Z1kkurat deleted the separate-kafka-module-ce2 branch July 7, 2023 14:58
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.

1 participant