Skip to content

Commit a5859c8

Browse files
committed
Cover range(DATE)->DATETIME case
1 parent feb8612 commit a5859c8

File tree

2 files changed

+8
-0
lines changed

2 files changed

+8
-0
lines changed

x-pack/plugin/esql-core/src/main/java/org/elasticsearch/xpack/esql/core/type/DataType.java

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -584,6 +584,13 @@ static Builder builder() {
584584
return new Builder();
585585
}
586586

587+
public boolean isDate() {
588+
return switch (this) {
589+
case DATETIME, DATE_NANOS -> true;
590+
default -> false;
591+
};
592+
}
593+
587594
/**
588595
* Named parameters with default values. It's just easier to do this with
589596
* a builder in java....

x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/enrich/EnrichLookupService.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -377,6 +377,7 @@ private static boolean rangeTypesCompatible(RangeType rangeType, DataType inputD
377377
return switch (rangeType) {
378378
case INTEGER, LONG -> inputDataType.isWholeNumber();
379379
case IP -> inputDataType == DataType.IP;
380+
case DATE -> inputDataType.isDate();
380381
default -> rangeType.isNumeric() == inputDataType.isNumeric();
381382
};
382383
}

0 commit comments

Comments
 (0)