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

[Debezium Server Name Mapper] Switch to Debezium 2.5, Quarkus to 3.0.0.Final #345

Merged
merged 2 commits into from
Feb 9, 2024

Conversation

ani-sha
Copy link
Member

@ani-sha ani-sha commented Feb 5, 2024

No description provided.

@ani-sha ani-sha force-pushed the dbz-server-name-mapper-update branch from 0f709a8 to 30331f2 Compare February 7, 2024 05:39
@jpechane
Copy link
Contributor

jpechane commented Feb 7, 2024

@Naros With this upgrade it seems the example no longer works. It seems that stream mapper bean is no longer resolved/found. Do you know if there was an change in the lookup mechanism?

@Naros
Copy link
Member

Naros commented Feb 9, 2024

Ya the issue here was that the pom was referencing the old Java EE cd-api rather than the Jakarta EE cdi-api that should be used now-a-days with the major shift from Java EE to Jakarta EE. I've added a commit that solves the issue and verified that the output, as seen here for the first entry on the query:

┌─ccranfor@fedora ~/github/Naros/debezium-examples-4/debezium-server-name-mapper (dbz-server-name-mapper-update)
└─$ docker compose exec pulsar bin/pulsar-admin broker-stats topics -i
{
    "public/default": {
        "0x00000000_0x40000000": {
            "persistent": {
                "persistent://public/default/superprefix.tutorial.inventory.products_on_hand": {

@Naros Naros force-pushed the dbz-server-name-mapper-update branch from 0bc0a33 to 6c3ed69 Compare February 9, 2024 22:40
@Naros
Copy link
Member

Naros commented Feb 9, 2024

Thanks @ani-sha, applied!

@Naros Naros merged commit f890bf7 into debezium:main Feb 9, 2024
1 check passed
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