Support postgresql kafka connector #23
Conversation
|
Atiq, Atiq Sayyed seem 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. |
Codecov Report
@@ Coverage Diff @@
## master #23 +/- ##
==========================================
+ Coverage 96.15% 96.19% +0.04%
==========================================
Files 7 7
Lines 260 263 +3
Branches 29 32 +3
==========================================
+ Hits 250 253 +3
Misses 10 10
Continue to review full report at Codecov.
|
| class JdbcSourceConnector extends SourceConnector { | ||
| private val logger = LoggerFactory.getLogger(classOf[JdbcSourceConnector]) | ||
|
|
||
| Class.forName("org.postgresql.Driver") |
There was a problem hiding this comment.
The driver should be loaded automatically, is it required to explicitly do this?
| case MySQL => connection.prepareStatement(s"CALL $storedProcedureName (@$incrementingVariableName := ?, @$batchSizeVariableName := ?)") | ||
| case MsSQL => connection.prepareStatement(s"EXECUTE $storedProcedureName @$incrementingVariableName = ?, @$batchSizeVariableName = ?") | ||
| case MySQL => connection.prepareStatement(s"CALL $storedProcedureName (@$incrementingVariableName := ?, @$batchSizeVariableName := ?)") | ||
| case PostgreSQL => connection.prepareStatement(s"SELECT * from $storedProcedureName (?, ?)") |
There was a problem hiding this comment.
Can we pass arguments by name?
3 participants
Ref #4