Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BREAKING] Implicit cast rules for integer literals, and parameterized ANY for binding #10194

Merged
merged 14 commits into from
Jan 10, 2024
Merged

[BREAKING] Implicit cast rules for integer literals, and parameterized ANY for binding #10194

Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
4beb4f9
Fix #10096 - prefer auto-casting to HUGEINT over DOUBLE to prevent lo…
Mytherin Jan 9, 2024
b494924
Clean up the way that the new ANY rework works by adding extra parame…
Mytherin Jan 9, 2024
596d72f
Revert constant changes
Mytherin Jan 9, 2024
815e2c4
Add the INTEGER_LITERAL class - similar to the STRING_LITERAL we use …
Mytherin Jan 9, 2024
d756f7d
Test fixes
Mytherin Jan 9, 2024
01cd568
CI fixes
Mytherin Jan 10, 2024
9d116c4
Merge with main
Mytherin Jan 10, 2024
f0d7883
Merge branch 'main' into issue10096
Mytherin Jan 10, 2024
e800b7a
Revert test
Mytherin Jan 10, 2024
e354fa6
Remove dead code
Mytherin Jan 10, 2024
32a226a
Add new test, revert unneeded changes
Mytherin Jan 10, 2024
b4f6748
Add test for types of combining integer literals
Mytherin Jan 10, 2024
36776aa
OP::Operation
Mytherin Jan 10, 2024
4b3316a
Fixes for Python tests
Mytherin Jan 10, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Remove dead code
  • Loading branch information
@Mytherin
Mytherin committed Jan 10, 2024
commit e354fa629bc045edc30c837e859e8240d41e9abd
7 changes: 0 additions & 7 deletions src/planner/expression/bound_cast_expression.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -81,13 +81,6 @@ unique_ptr<Expression> AddCastToTypeInternal(unique_ptr<Expression> expr, const
auto &def = expr->Cast<BoundDefaultExpression>();
def.return_type = target_type;
}
if (expr->return_type.id() == LogicalTypeId::STRING_LITERAL) {
throw InternalException("FIXME - does this ever happen...?");
// when adding a cast to a string literal we need to transform the constant back into a VARCHAR
auto &constant = expr->Cast<BoundConstantExpression>();
constant.value = Value(StringValue::Get(constant.value));
constant.return_type = LogicalType::VARCHAR;
}
if (!target_type.IsValid()) {
return expr;
}
Expand Down