andrew4699 opened a new pull request, #412: URL: https://github.com/apache/polaris/pull/412
# Description TestFileIOs could be forced to throw an exception when calling getLength. Unfortunately this is hard to use because TestFileIOs also contain functionality for measuring file sizes. This meant that any supplied getLength exception would result in the construction of an InputFile to immediately fail instead of the exception being deferred to later. This PR makes the TestFileIO use the non-exception-throwing-InputFile for measurement. Unfortunately the previous tests were relying on getLength being called at some point during the workloads but Iceberg seems to rarely call getLength, thus I just wrote a separate test to validate the functionality. ## Type of change - [X] Bug fix (non-breaking change which fixes an issue) # How Has This Been Tested? - [X] Existing tests # Checklist: Please delete options that are not relevant. - [X] I have performed a self-review of my code - [X] I have commented my code, particularly in hard-to-understand areas - [X] My changes generate no new warnings -- 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]
