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

Update nullable columns to NULL is not reported correctly #14

Closed
Oli180369 opened this issue Mar 4, 2021 · 5 comments
Closed

Update nullable columns to NULL is not reported correctly #14

Oli180369 opened this issue Mar 4, 2021 · 5 comments

Comments

@Oli180369
Copy link
Contributor

If a nullable column is updated with NULL the new value is not reported as NULL, much more the old value is reported as the new value.

@Oli180369 Oli180369 changed the title Update nullable columns to NULL ist not reported correctly Update nullable columns to NULL is not reported correctly Mar 4, 2021
@thake
Copy link
Owner

thake commented Mar 4, 2021 via email

Oli180369 added a commit to Oli180369/logminer-kafka-connect that referenced this issue Mar 4, 2021
@Oli180369
Copy link
Contributor Author

Seems that we have an issue with the logminer itself

@Oli180369
Copy link
Contributor Author

We checked the tables on which the logminer-kafka-connect component is working. These tables were filled correctly. There has to be an issue with the component. We try to improve the test

@Oli180369
Copy link
Contributor Author

It seems that we have a problem within the ConnectSchemaFactrory at line 99 where the after struct is enriched by before values for absent keys. At this point the component can not distinguish an absence from an NULL-Update because the Hashmap will return NULL in both cases. So the after struct is always enriched by the before struct for columns which are updated to NULL.
I was not able to provide a test for this issue so far, but I hope you have understood.

thake added a commit that referenced this issue Mar 9, 2021
@thake
Copy link
Owner

thake commented Mar 9, 2021

@Oli180369 Thanks for this patch! A tricky bug.. ;)

I'll prepare a new release just now.

@thake thake closed this as completed Mar 9, 2021
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

2 participants