skytin1004 commented on PR #7655:
URL: https://github.com/apache/iceberg/pull/7655#issuecomment-1571676337
@nastra Thank you for your review. I modified the code based on your review
:)
I think the casting should be left as it is, because the test will fail if
the units are different.
When I remove the casting, the test failed because the test is comparing an
int value to a long value.
So I think it's correct to cast the i in the long type.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
