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

Reduced the list of dependencies packaged in the Kafka Connector distro #854

Merged
merged 1 commit into from
Sep 20, 2020
Merged

Reduced the list of dependencies packaged in the Kafka Connector distro #854

merged 1 commit into from
Sep 20, 2020

Conversation

EricWittmann
Copy link
Member

See: #497

@EricWittmann
Copy link
Member Author

@jpechane What do you think? :)

@jpechane
Copy link
Contributor

@EricWittmann Great job, thanks a lot! I don't see any other unnecessary file in it and I've successfully tested the converter.

For product though I'd recommend to consider using the Kafka Connect Jackson version instead of bringing your own (as mentioned before it works with it).

In what varison (and schedule) do you plan to release this distro? We'd update our dependencies immediately.

@EricWittmann
Copy link
Member Author

An upstream release (1.3.1.Final) will be done shortly. I don't have a schedule for product. Q4 is possible.

@EricWittmann EricWittmann merged commit 93b9d21 into Apicurio:master Sep 20, 2020
@EricWittmann
Copy link
Member Author

@jpechane Upstream 1.3.1.Final has been released.

@jpechane
Copy link
Contributor

Thanks, unfortunately somethin went wrong #864

@EricWittmann
Copy link
Member Author

OK thanks @jpechane that's unfortunate. We definitely need a good integration test for this distro. @famartinrh is working on that!

@famarting
Copy link
Contributor

@EricWittmann I successfully implemented a test for this in our k8s testsuite and I'm able to reproduce the error... https://gist.github.com/famartinrh/8db30fcb680e14021903a414aeed2c61 here a gist with the logs of the debezium pod in case it helps

@EricWittmann EricWittmann deleted the issues/497-2 branch September 24, 2020 20:57
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