kafka-connect-jdbc
kafka-connect-jdbc copied to clipboard
Support for multiple custom queries and timestamps without timezone
This is a proposition relative to issues #113 and #103 that I faced on my project. It gives good results on the client setup : large Oracle DB with a lot of views needed and no timezone in timestamps (Oracle DATE type). Multiple custom queries can be configured as views definitions in table.whitelist, without needing rights to add Views to the schema.
Also added support for Postgres blobs in PostgreSql Sink Dialect (BYTEA). See Readme.md for a complete description.
I tried to adhere to existing basecode to add these features cleanly, even if some refactoring would now certainly be necessary (too much arguments in TimestampIncrementingTableQuerier for instance). Also, a better solution concerning escaping in config would need to close previously KAFKA-4524
As written in Readme, I added some unit tests covering these features. Edit 18/03/19 : with Jenkins build reports accessible now, updated the fork to pass all the tests.
It looks like @math-g hasn't signed our Contributor License Agreement, yet.
The purpose of a CLA is to ensure that the guardian of a project's outputs has the necessary ownership or grants of rights over all contributions to allow them to distribute under the chosen licence. Wikipedia
You can read and sign our full Contributor License Agreement here.
Once you've signed reply with [clabot:check]
to prove it.
Appreciation of efforts,
clabot
[clabot:check]
@confluentinc It looks like @math-g just signed our Contributor License Agreement. :+1:
Always at your service,
clabot
Thanks for the PR! I've tagged a few engineers to review further.
This would be a great feature for scaling connector. Pity it hasn't been completed.
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
0 out of 2 committers have signed the CLA.
:x: Mathieu Grelier
:x: math-g
Mathieu Grelier seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.