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

Installation guide, updated on how to install Debezium in Kafka Connect environment #206

Open
wants to merge 10 commits into
base: develop
Choose a base branch
from
Open

Conversation

jpsoroulas
Copy link
Contributor

@jpsoroulas jpsoroulas commented Oct 3, 2018

The "Installing Debezium" document has been updated with details on how to install Debezium in Kafka Connect environment

The contents of https://gist.github.com/jpsoroulas/30e9537138ca62a79fe261cff7ceb716#2-debezium-connector-deployment-at-confluent-platform have been merged into the https://debezium.io/docs/install/

The document has reference to the "Postgresql, logical decoding output plugin installation" (pending pull number #204)

Some details can be found at the discussion with @gunnarmorling :
https://groups.google.com/forum/#!topic/debezium/h5B52YTkGR4

@jpsoroulas jpsoroulas changed the title Updated on how to install Debezium in Kafka Connect environment Debezium installation guide, updated on how to install Debezium in Kafka Connect environment Oct 3, 2018
@jpsoroulas jpsoroulas changed the title Debezium installation guide, updated on how to install Debezium in Kafka Connect environment Installation guide, updated on how to install Debezium in Kafka Connect environment Oct 3, 2018
@gunnarmorling
Copy link
Member

Hey @jpsoroulas, just wanted to let you know that this PR (and your other one) isn't forgotten. I'm in middle of preparing a Debezium workshop right now, so I'm a bit constrained time-wise. I'll get back to this asap. Thanks again for your efforts and sorry for the delay.

@jpsoroulas
Copy link
Contributor Author

No problem @gunnarmorling , take your time,
good luck with the workshop!

@gunnarmorling
Copy link
Member

Thanks for the update to this PR, @jpsoroulas! Big sorry for the delay with this PR, it fell a bit through the cracks :( I'll get back to it now.

@jpsoroulas
Copy link
Contributor Author

No problem @gunnarmorling!
When you're ready, please do not forget to also check PR #204.

@jpsoroulas
Copy link
Contributor Author

Hi @gunnarmorling,
I hope I am not imposing on you but I think it is time to close the PR.
It is around five months old and I constantly try to keep it updated.
If you are still interesting, lets make the changes and close it.
By the way, nice presentation at DevNation Live.

@gunnarmorling
Copy link
Member

gunnarmorling commented Feb 21, 2019

Hey @jpsoroulas, you're not imposing at all. I really feel bad about how this one was handled (or rather not) and all I can offer is a sincere apology. It constantly got put into the back seat due to other things going on, there's just too much.

I'm definitely still interested and would love to get it done. Right now it doesn't seem to apply, so could you rebase it one more time and update the PR? I'll then apply some changes (e.g. adding a link to the OpenShift/Strimzi instructions) and it should be good to go. Thanks a lot, and sorry again. Looking forward to getting this in soon.

@jpsoroulas
Copy link
Contributor Author

I am glad to hear that @gunnarmorling!
I merge the branch and I also update the versions of the used libraries

 into instdoc-connectenv

# Conflicts:
#	_partials/install.html.haml
@jpsoroulas
Copy link
Contributor Author

Hi @gunnarmorling,
the PR is pending for a long time and since the debezium site has been majorly refactored, I find impractical to keep this PR open. I updated my gist for debezium installation according to the content of my PR.

If you still finding the article interesting you can added at the debezium online documentation resources.

keep good working!

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.

2 participants