JeonDaehong commented on issue #13921: URL: https://github.com/apache/iceberg/issues/13921#issuecomment-3230273542
> we should review all the usage of `IntStream.rangeClosed`, there are not that many. > For `rangeClosed`, both bounds are inclusive. For `range`, start is inclusive, end is exclusive. Just making sure `rangeClosed` is what we want @kevinjqliu So we need to review the places where rangeClosed is used, and if there are cases where range should be used instead, we should update them to range. Thank you! -- 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]
