Skip to content

Commit a0fd371

Browse files
committed
ignore case added
1 parent 5fb8e4d commit a0fd371

File tree

2 files changed

+32
-1
lines changed

2 files changed

+32
-1
lines changed

data-loader/core/src/main/java/com/scalar/db/dataloader/core/util/ColumnUtils.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -83,7 +83,7 @@ public static Column<?> createColumnFromValue(
8383
DataType dataType, ColumnInfo columnInfo, @Nullable String value)
8484
throws ColumnParsingException {
8585
String columnName = columnInfo.getColumnName();
86-
if (value != null && !dataType.equals(DataType.TEXT) && value.equals("null")) {
86+
if (value != null && !dataType.equals(DataType.TEXT) && value.equalsIgnoreCase("null")) {
8787
value = null;
8888
}
8989
try {

data-loader/core/src/test/java/com/scalar/db/dataloader/core/util/ColumnUtilsTest.java

Lines changed: 31 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -242,27 +242,49 @@ void createColumnFromValue_valueIsLowercaseNull_shouldReturnNullColumn()
242242
// Integer type
243243
Column<?> intColumn = ColumnUtils.createColumnFromValue(DataType.INT, columnInfo, "null");
244244
assertEquals(IntColumn.ofNull(columnName), intColumn);
245+
intColumn = ColumnUtils.createColumnFromValue(DataType.INT, columnInfo, "Null");
246+
assertEquals(IntColumn.ofNull(columnName), intColumn);
245247

246248
// Double type
247249
Column<?> doubleColumn = ColumnUtils.createColumnFromValue(DataType.DOUBLE, columnInfo, "null");
248250
assertEquals(DoubleColumn.ofNull(columnName), doubleColumn);
251+
doubleColumn = ColumnUtils.createColumnFromValue(DataType.DOUBLE, columnInfo, "NULL");
252+
assertEquals(DoubleColumn.ofNull(columnName), doubleColumn);
249253

250254
// Boolean type
251255
Column<?> boolColumn = ColumnUtils.createColumnFromValue(DataType.BOOLEAN, columnInfo, "null");
252256
assertEquals(BooleanColumn.ofNull(columnName), boolColumn);
253257

258+
boolColumn = ColumnUtils.createColumnFromValue(DataType.BOOLEAN, columnInfo, "nuLL");
259+
assertEquals(BooleanColumn.ofNull(columnName), boolColumn);
260+
254261
// Date type
255262
Column<?> dateColumn = ColumnUtils.createColumnFromValue(DataType.DATE, columnInfo, "null");
256263
assertEquals(DateColumn.ofNull(columnName), dateColumn);
264+
dateColumn = ColumnUtils.createColumnFromValue(DataType.DATE, columnInfo, "NULL");
265+
assertEquals(DateColumn.ofNull(columnName), dateColumn);
257266

258267
// Time type
259268
Column<?> timeColumn = ColumnUtils.createColumnFromValue(DataType.TIME, columnInfo, "null");
260269
assertEquals(TimeColumn.ofNull(columnName), timeColumn);
261270

271+
timeColumn = ColumnUtils.createColumnFromValue(DataType.TIME, columnInfo, "nuLL");
272+
assertEquals(TimeColumn.ofNull(columnName), timeColumn);
273+
262274
// Timestamp type
263275
Column<?> timestampColumn =
264276
ColumnUtils.createColumnFromValue(DataType.TIMESTAMP, columnInfo, "null");
265277
assertEquals(TimestampColumn.ofNull(columnName), timestampColumn);
278+
timestampColumn = ColumnUtils.createColumnFromValue(DataType.TIMESTAMP, columnInfo, "NULL");
279+
assertEquals(TimestampColumn.ofNull(columnName), timestampColumn);
280+
281+
// Timestamp type
282+
Column<?> timestamprtzColumn =
283+
ColumnUtils.createColumnFromValue(DataType.TIMESTAMPTZ, columnInfo, "null");
284+
assertEquals(TimestampTZColumn.ofNull(columnName), timestamprtzColumn);
285+
timestamprtzColumn =
286+
ColumnUtils.createColumnFromValue(DataType.TIMESTAMPTZ, columnInfo, "Null");
287+
assertEquals(TimestampTZColumn.ofNull(columnName), timestamprtzColumn);
266288
}
267289

268290
/**
@@ -277,5 +299,14 @@ void createColumnFromValue_valueIsNullString_shouldRemainLiteralForTextType()
277299

278300
Column<?> textCol = ColumnUtils.createColumnFromValue(DataType.TEXT, columnInfo, "null");
279301
assertEquals(TextColumn.of(columnName, "null"), textCol);
302+
303+
textCol = ColumnUtils.createColumnFromValue(DataType.TEXT, columnInfo, "NULL");
304+
assertEquals(TextColumn.of(columnName, "NULL"), textCol);
305+
306+
textCol = ColumnUtils.createColumnFromValue(DataType.TEXT, columnInfo, "Null");
307+
assertEquals(TextColumn.of(columnName, "Null"), textCol);
308+
309+
textCol = ColumnUtils.createColumnFromValue(DataType.TEXT, columnInfo, "nuLL");
310+
assertEquals(TextColumn.of(columnName, "nuLL"), textCol);
280311
}
281312
}

0 commit comments

Comments
 (0)