Skip to content

Commit

Permalink
Add Calcite follow-up ticket
Browse files Browse the repository at this point in the history
  • Loading branch information
rubenada committed Apr 15, 2024
1 parent b25a0be commit f3b8a78
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -355,7 +355,7 @@ public static Expression convert(Expression operand, Type fromType,
return operand;
}

// TODO consider using Expressions#convertChecked to throw exception in case of overflow
// TODO use Expressions#convertChecked to throw exception in case of overflow (CALCITE-6366)

// E.g. from "Short" to "int".
// Generate "x.intValue()".
Expand Down
2 changes: 1 addition & 1 deletion core/src/test/java/org/apache/calcite/test/JdbcTest.java
Original file line number Diff line number Diff line change
Expand Up @@ -8466,7 +8466,7 @@ private void checkGetTimestamp(Connection con) throws SQLException {
.returnsUnordered("VAL=2500.55");
}

@Disabled
@Disabled("CALCITE-6366")
@Test void bindOverflowingTinyIntParameter() {
final String sql =
"with cte as (select cast(300 as smallint) as empid)"
Expand Down

0 comments on commit f3b8a78

Please sign in to comment.