mihaibudiu commented on code in PR #3743:
URL: https://github.com/apache/calcite/pull/3743#discussion_r1567724269
##########
testkit/src/main/java/org/apache/calcite/test/IntervalTest.java:
##########
@@ -870,8 +870,6 @@ public void subTestIntervalYearNegative() {
// Field value out of range
// (default, explicit default, alt, neg alt, max, neg max)
- f.wholeExpr("INTERVAL '100' YEAR")
Review Comment:
I have added the removed tests back as positive tests, but I have kept them
in the same place in the code.
I could move them to separate functions too if that's clearer.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
