Open gallejesus opened 4 months ago
Hello! Have you been abel to take a look at the changes or want a bit more details on them? :)
Hi @gallejesus, apologies I've been on leave for the past few weeks and I miss some of these notifications. Can review this now.
We have created this PR to fix some issues we have found when workig with ClickHouse target.
We can discuss them more in depth but main changes include the addition of tests covering more data types, explicit override of sql data type conversion and manage of nulls and keeping original name from source.
Let me know what you think of those :)
Thank you!