xtern commented on code in PR #7630:
URL: https://github.com/apache/ignite-3/pull/7630#discussion_r2827210049
##########
modules/table/src/integrationTest/java/org/apache/ignite/internal/table/ItRecordBinaryViewApiTest.java:
##########
@@ -328,7 +345,7 @@ public void validateSchema(BinTestCase testCase) {
RecordView<Tuple> tbl = testCase.view();
- Tuple keyTuple0 = Tuple.create().set("id", 0).set("id1", 0);
+ Tuple keyTuple0 = Tuple.create().set("id",
Double.MAX_VALUE).set("id1", 0);
Review Comment:
It's related - previously we didn't support implicit conversion the test
verifies error message when conversion is ot possible
previous message "expected=INT64, actual=INT32"
INT32-> INT64 is always possible that's way the value was changed to double
--
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]
