Skip to content

Commit 4b3c0d3

Browse files
amorynanyiguolei
authored andcommitted
[fix](array) fix arr funcs with ret which BE should eginore precision because of deduce that the FE planner cannot infer (#57018)
### What problem does this PR solve? Issue Number: close #xxx Related PR: #xxx Problem Summary: ### Release note None ### Check List (For Author) - Test <!-- At least one of them must be included. --> - [ ] Regression test - [ ] Unit Test - [ ] Manual test (add detailed scripts or steps below) - [ ] No need to test or manual test. Explain why: - [ ] This is a refactor/code format and no logic has been changed. - [ ] Previous test can cover this change. - [ ] No code files have been changed. - [ ] Other reason <!-- Add your reason? --> - Behavior changed: - [ ] No. - [ ] Yes. <!-- Explain the behavior change --> - Does this need documentation? - [ ] No. - [ ] Yes. <!-- Add document PR link here. eg: apache/doris-website#1214 --> ### Check List (For Reviewer who merge this PR) - [ ] Confirm the release note - [ ] Confirm test cases - [ ] Confirm document - [ ] Add branch pick label <!-- Add branch pick label that this PR should merge into -->
1 parent 9438031 commit 4b3c0d3

File tree

3 files changed

+3
-3
lines changed

3 files changed

+3
-3
lines changed

be/src/vec/functions/array/function_array_binary.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ class FunctionArrayBinary : public IFunction {
4343
assert_cast<const DataTypeArray&>(*(arguments[0])).get_nested_type());
4444
auto right_nested_type = remove_nullable(
4545
assert_cast<const DataTypeArray&>(*(arguments[1])).get_nested_type());
46-
DCHECK(left_nested_type->equals(*right_nested_type))
46+
DCHECK(left_nested_type->equals_ignore_precision(*right_nested_type))
4747
<< "data type " << arguments[0]->get_name() << " not equal with "
4848
<< arguments[1]->get_name();
4949
return Impl::get_return_type(arguments);

be/src/vec/functions/array/function_array_contains_all.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ class FunctionArrayContainsAll : public IFunction {
5353
assert_cast<const DataTypeArray&>(*left_data_type).get_nested_type());
5454
auto right_nested_type = remove_nullable(
5555
assert_cast<const DataTypeArray&>(*right_data_type).get_nested_type());
56-
DCHECK(left_nested_type->equals(*right_nested_type))
56+
DCHECK(left_nested_type->equals_ignore_precision(*right_nested_type))
5757
<< "data type " << arguments[0]->get_name() << " not equal with "
5858
<< arguments[1]->get_name();
5959
return std::make_shared<DataTypeUInt8>();

be/src/vec/functions/array/function_arrays_overlap.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -151,7 +151,7 @@ class FunctionArraysOverlap : public IFunction {
151151
assert_cast<const DataTypeArray&>(*left_data_type).get_nested_type());
152152
auto right_nested_type = remove_nullable(
153153
assert_cast<const DataTypeArray&>(*right_data_type).get_nested_type());
154-
DCHECK(left_nested_type->equals(*right_nested_type))
154+
DCHECK(left_nested_type->equals_ignore_precision(*right_nested_type))
155155
<< "data type " << arguments[0]->get_name() << " not equal with "
156156
<< arguments[1]->get_name();
157157
return make_nullable(std::make_shared<DataTypeUInt8>());

0 commit comments

Comments
 (0)