ygerzhedovich commented on code in PR #5707: URL: https://github.com/apache/ignite-3/pull/5707#discussion_r2067995059
########## modules/sql-engine/src/integrationTest/sql/group1/types/time/time_parsing.test: ########## @@ -28,84 +28,88 @@ SELECT '14:42:04.999000'::TIME(6)::VARCHAR ---- 14:42:04.999 -query T +statement error: Invalid TIME value, '1' SELECT '1'::TIME ----- -01:00:00 -query T +statement error: Invalid TIME value, '11' SELECT '11'::TIME ----- -11:00:00 -statement error +statement error: Invalid TIME value, '11:' SELECT '11:'::TIME -query T +statement error: Invalid TIME value, '11:1' SELECT '11:1'::TIME ----- -11:01:00 -statement error +statement error: Invalid TIME value, '11:01' +SELECT '11:01'::TIME + +statement error: Value of HOUR field is out of range in '25:42:04.500' +SELECT '25:42:04.500'::TIME(3) + +statement error: Invalid TIME value SELECT '23:60:00'::TIME ----- -# https://issues.apache.org/jira/browse/IGNITE-25000 Negative time components / out of range values -skipif ignite3 -statement error +statement error: Invalid TIME value, '23:00:60' SELECT '23:00:60'::TIME -# https://issues.apache.org/jira/browse/IGNITE-25000 Negative time components / out of range values -skipif ignite3 -statement error +statement error: Invalid TIME value, '22:60:60' SELECT '22:60:60'::TIME -statement error +statement error: Invalid TIME value, '23:59:60' SELECT '23:59:60'::TIME ----- -# https://issues.apache.org/jira/browse/IGNITE-25000 Negative time components / out of range values -skipif ignite3 -statement error +statement error: Invalid TIME value, '100:42:04.500' +SELECT '100:42:04.500'::TIME(3) Review Comment: It's the same type of error as on line 47, however, the message has different format -- 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: notifications-unsubscr...@ignite.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
