search

exasol / kafka-connector-extension

Exasol Kafka Extension for accessing Apache Kafka
MIT License
4 stars 7 forks source link

Emit null for all columns if Kafka record has a null value #31

Closed jwarlander closed 3 years ago

jwarlander commented 3 years ago

Fixes #29.

This is quite possibly not The Right Way to solve the issue, but it seems to work. I wasn't able to figure out how to test sending in a null value in the integration tests, so any help is appreciated!

morazow commented 3 years ago

Hello @jwarlander ,

Thanks for the feedback and PR! We really appreciate it!

I am going to have review soon.

morazow commented 3 years ago

Hello @jwarlander,

I have added minor change and test for records with null value emit.

This now looks good! Should we merge it? ~Could you please make the PR ready?~ Okay, I could change it also :)