Skip to content

Commit 7222899

Browse files
authored
Merge pull request #141 from data-integrations/fix/snapshot-handling
Correctly mark record if it is part of snapshot or not for mysql.
2 parents 0cc46fe + 514bca7 commit 7222899

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

mysql-delta-plugins/src/main/java/io/cdap/delta/mysql/MySqlRecordConsumer.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -126,7 +126,7 @@ public void accept(SourceRecord sourceRecord) {
126126
// This should not happen, 'source' is a mandatory field in sourceRecord from debezium
127127
return;
128128
}
129-
boolean isSnapshot = Boolean.TRUE.equals(deltaOffset.get(MySqlConstantOffsetBackingStore.SNAPSHOT));
129+
boolean isSnapshot = Boolean.parseBoolean(deltaOffset.get(MySqlConstantOffsetBackingStore.SNAPSHOT));
130130
// If the map is empty, we should read all DDL/DML events and columns of all tables
131131
boolean readAllTables = sourceTableMap.isEmpty();
132132

sqlserver-delta-plugins/src/main/java/io/cdap/delta/sqlserver/SqlServerOffset.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@ public class SqlServerOffset {
4848
}
4949

5050
boolean isSnapshot() {
51-
return Boolean.TRUE.equals(isSnapshot);
51+
return isSnapshot;
5252
}
5353

5454
void setSnapshotTables(Set<String> snapshotTables) {

0 commit comments

Comments
 (0)