Skip to content

Commit

Permalink
[Bug][chunjun-connectors] jdbc's properties not work where jdbc's rea…
Browse files Browse the repository at this point in the history
…der has set properties
  • Loading branch information
AliceXiaoLu committed Feb 21, 2024
1 parent 38fed40 commit 0f91dd5
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -89,7 +89,7 @@ public JdbcSourceFactory(
if (StringUtils.isBlank(jdbcConfig.getIncreColumn())) jdbcConfig.setPolling(false);
jdbcConfig.setColumn(syncConfig.getReader().getFieldList());

Properties properties = syncConfig.getWriter().getProperties("properties", null);
Properties properties = syncConfig.getReader().getProperties("properties", null);
jdbcConfig.setProperties(properties);

setDefaultSplitStrategy(jdbcConfig);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ public void toStringShouldReturnAStringWithAllTheFields() {
baseFileConf.setNextCheckRows(5000L);

String expected =
"BaseFileConfig(fromLine=1, path=path, fileName=fileName, writeMode=writeMode, compress=compress, encoding=UTF-8, maxFileSize=1024, nextCheckRows=5000, suffix=null)";
"BaseFileConfig(fromLine=1, path=path, fileName=fileName, writeMode=writeMode, compress=compress, encoding=UTF-8, maxFileSize=1024, nextCheckRows=5000, suffix=null, jobIdentifier=)";

assertEquals(expected, baseFileConf.toString());
}
Expand Down

0 comments on commit 0f91dd5

Please sign in to comment.