confluentinc / kafka-connect-jdbc

Kafka Connect connector for JDBC-compatible databases
Other
1.01k stars 953 forks source link

CUP-1584 Stored Procedure support #1410

Open pablodc00 opened 2 months ago

pablodc00 commented 2 months ago

Problem

The connector does not support reading from Stored Procedures

Solution

Added support for reading from Stored Procedure

Does this solution apply anywhere else?
If yes, where?

Test Strategy

Testing done:

Release Plan

cla-assistant[bot] commented 2 months ago

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

cla-assistant[bot] commented 2 months ago

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

anastasiya-yakushina commented 2 months ago

I guess there is a little confusion in terms of modes. My suggestion is to leave the existing display modes, which are "bulk, timestamp, incrementing and timestamp+incrementing" and add a new querier mode to have "query, table and storedprocedure". In our case bulk with storedprocedure should work fine