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

Next release version of sdk-go will require kafka_sarama users to update their module path #938

Open
dnwe opened this issue Sep 18, 2023 · 4 comments

Comments

@dnwe
Copy link

dnwe commented Sep 18, 2023

πŸ‘‹πŸ» hi, Sarama maintainer here

PR #930 updated the module path of Sarama from github.com/Shopify/sarama to github.com/IBM/sarama β€” which long term is the correct thing to do as Shopify asked us to take over ownership of the library going forward and we've already made several releases

However, the PR was done under the existing v2 module path of github.com/cloudevents/sdk-go/protocol/kafka_sarama/v2, yet it is a breaking change in terms of callers of the library who will need to update their import path of Sarama before bumping their cloudevents/sdk-go module. It's not yet a problem, as a new release tag hasn't been published, but it would be worth thinking about what you plan to do here as it would really need to be done either under a major semver v3 release or under a new protocol submodule, leaving the old kafka_sarama one in-place

@embano1
Copy link
Member

embano1 commented Sep 19, 2023

Thx for flagging this @dnwe ! And you're right, we should have used a new protocol submodule to avoid the breaking change. IMHO this change doesn't justify a v3 of the SDK (besides the fact that semver rules require it). So IMHO we have to options:

  1. fix the current behavior, i.e., revert and reapply the change to a new v3 submodule
  2. Announce this change as a breaking change in our release notes (which doesn't prevent users from running into the issue, but would be documented).

My understanding is that only users importing kafka_sarama would be affected by (2)?

cc/ @lionelvillard @duglin for further input/thoughts

@dnwe
Copy link
Author

dnwe commented Sep 19, 2023

Correct yes, it would only effect people importing and calling into kafka_sarama itself. I agree that a breaking change release note is probably sufficient. Ultimately there won't be any more releases on the github.com/Shopify/sarama import path so the consumers will have to update their module path at some point.

If you can tag a new point release of v2 then I can always submit a few PRs to the immediate importers to sync them up

@embano1
Copy link
Member

embano1 commented Sep 19, 2023

SGTM

@duglin
Copy link
Contributor

duglin commented Sep 19, 2023

To be clear.... you guys are suggesting option 2, right? If so, I'm ok with that.

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

No branches or pull requests

3 participants