Skip to content

Commit

Permalink
uniform variable naming convention
Browse files Browse the repository at this point in the history
  • Loading branch information
vinlee19 committed Mar 20, 2024
1 parent 5de2d5b commit e4445af
Show file tree
Hide file tree
Showing 5 changed files with 21 additions and 21 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -71,7 +71,7 @@ public class JsonDebeziumSchemaSerializer implements DorisRecordSerializer<Strin
private String targetDatabase;
private String targetTablePrefix;
private String targetTableSuffix;
private boolean ignoreInCompatible;
private boolean ignoreIncompatible;
private JsonDebeziumDataChange dataChange;
private JsonDebeziumSchemaChange schemaChange;
private final Set<String> initTableSet = new HashSet<>();
Expand Down Expand Up @@ -138,14 +138,14 @@ public JsonDebeziumSchemaSerializer(
String targetDatabase,
String targetTablePrefix,
String targetTableSuffix,
boolean ignoreInCompatible) {
boolean ignoreIncompatible) {
this(dorisOptions, pattern, sourceTableName, newSchemaChange, executionOptions);
this.tableMapping = tableMapping;
this.tableProperties = tableProperties;
this.targetDatabase = targetDatabase;
this.targetTablePrefix = targetTablePrefix;
this.targetTableSuffix = targetTableSuffix;
this.ignoreInCompatible = ignoreInCompatible;
this.ignoreIncompatible = ignoreIncompatible;
init();
}

Expand All @@ -163,7 +163,7 @@ private void init() {
ignoreUpdateBefore,
targetTablePrefix,
targetTableSuffix,
ignoreInCompatible);
ignoreIncompatible);
this.schemaChange =
newSchemaChange
? new JsonDebeziumSchemaChangeImplV2(changeContext)
Expand Down Expand Up @@ -227,7 +227,7 @@ public static class Builder {
private String targetDatabase;
private String targetTablePrefix = "";
private String targetTableSuffix = "";
private boolean ignoreInCompatible = true;
private boolean ignoreIncompatible = true;

public JsonDebeziumSchemaSerializer.Builder setDorisOptions(DorisOptions dorisOptions) {
this.dorisOptions = dorisOptions;
Expand Down Expand Up @@ -283,8 +283,8 @@ public Builder setTargetTableSuffix(String tableSuffix) {
return this;
}

public Builder setIgnoreInCompatible(boolean ignoreInCompatible) {
this.ignoreInCompatible = ignoreInCompatible;
public Builder setIgnoreIncompatible(boolean ignoreInCompatible) {
this.ignoreIncompatible = ignoreInCompatible;
return this;
}

Expand All @@ -300,7 +300,7 @@ public JsonDebeziumSchemaSerializer build() {
targetDatabase,
targetTablePrefix,
targetTableSuffix,
ignoreInCompatible);
ignoreIncompatible);
}
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ public class JsonDebeziumChangeContext implements Serializable {
private final boolean ignoreUpdateBefore;
private String targetTablePrefix;
private String targetTableSuffix;
private boolean ignoreInCompatible;
private boolean ignoreIncompatible;

public JsonDebeziumChangeContext(
DorisOptions dorisOptions,
Expand All @@ -54,7 +54,7 @@ public JsonDebeziumChangeContext(
boolean ignoreUpdateBefore,
String targetTablePrefix,
String targetTableSuffix,
boolean ignoreInCompatible) {
boolean ignoreIncompatible) {
this.dorisOptions = dorisOptions;
this.tableMapping = tableMapping;
this.sourceTableName = sourceTableName;
Expand All @@ -66,7 +66,7 @@ public JsonDebeziumChangeContext(
this.ignoreUpdateBefore = ignoreUpdateBefore;
this.targetTablePrefix = targetTablePrefix;
this.targetTableSuffix = targetTableSuffix;
this.ignoreInCompatible = ignoreInCompatible;
this.ignoreIncompatible = ignoreIncompatible;
}

public DorisOptions getDorisOptions() {
Expand Down Expand Up @@ -113,7 +113,7 @@ public String getTargetTableSuffix() {
return targetTableSuffix;
}

public boolean isIgnoreInCompatible() {
return ignoreInCompatible;
public boolean isIgnoreIncompatible() {
return ignoreIncompatible;
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -75,7 +75,7 @@ public class JsonDebeziumSchemaChangeImplV2 extends JsonDebeziumSchemaChange {
private String targetDatabase;
private String targetTablePrefix;
private String targetTableSuffix;
private boolean ignoreInCompatible;
private boolean ignoreIncompatible;
private final Set<String> filledTables = new HashSet<>();

public JsonDebeziumSchemaChangeImplV2(JsonDebeziumChangeContext changeContext) {
Expand All @@ -96,7 +96,7 @@ public JsonDebeziumSchemaChangeImplV2(JsonDebeziumChangeContext changeContext) {
changeContext.getTargetTableSuffix() == null
? ""
: changeContext.getTargetTableSuffix();
this.ignoreInCompatible = changeContext.isIgnoreInCompatible();
this.ignoreIncompatible = changeContext.isIgnoreIncompatible();
}

@Override
Expand Down Expand Up @@ -128,7 +128,7 @@ public boolean schemaChange(JsonNode recordRoot) {

boolean ignoreSchema =
DatabaseSync.shouldIgnoreSchemaIncompatible(
ignoreInCompatible,
ignoreIncompatible,
tableSchema.getTable(),
tableSchema.getFields());
// If ignoreInCompatible is false, an exception is thrown when the source table
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -117,7 +117,7 @@ private static void syncDatabase(
String multiToOneTarget = params.get("multi-to-one-target");
boolean createTableOnly = params.has("create-table-only");
boolean ignoreDefaultValue = params.has("ignore-default-value");
boolean ignoreIncompatible = params.getBoolean("ignore-incompatible",true);
boolean ignoreIncompatible = params.getBoolean("ignore-incompatible", true);
boolean singleSink = params.has("single-sink");

Preconditions.checkArgument(params.has("sink-conf"));
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -83,7 +83,7 @@ public abstract class DatabaseSync {
protected String tablePrefix;
protected String tableSuffix;
protected boolean singleSink;
protected boolean ignoreInCompatible = true;
protected boolean ignoreIncompatible = true;
private final Map<String, String> tableMapping = new HashMap<>();

public abstract void registerDriver() throws SQLException;
Expand Down Expand Up @@ -135,7 +135,7 @@ public void build() throws Exception {
// mismatches the Doris schema.
// If it's true, incompatible schemas are skipped, and a warning log is printed.
if (shouldIgnoreSchemaIncompatible(
ignoreInCompatible, schema.getTableName(), schema.getFields())) {
ignoreIncompatible, schema.getTableName(), schema.getFields())) {
continue;
}

Expand Down Expand Up @@ -354,7 +354,7 @@ public DorisRecordSerializer<String> buildSchemaSerializer(
.setTargetDatabase(database)
.setTargetTablePrefix(tablePrefix)
.setTargetTableSuffix(tableSuffix)
.setIgnoreInCompatible(ignoreInCompatible)
.setIgnoreIncompatible(ignoreIncompatible)
.build();
}

Expand Down Expand Up @@ -550,7 +550,7 @@ public DatabaseSync setTableSuffix(String tableSuffix) {
}

public DatabaseSync setIgnoreIncompatible(boolean ignoreIncompatible) {
this.ignoreInCompatible = ignoreIncompatible;
this.ignoreIncompatible = ignoreIncompatible;
return this;
}

Expand Down

0 comments on commit e4445af

Please sign in to comment.