NeQuissimus edited a comment on pull request #3293: URL: https://github.com/apache/iceberg/pull/3293#issuecomment-947959498
@rdblue right, that's what I figured. Does that not mean the entire test is pointless now? The folder/files will not exist before or after calling `abort` since the (now) lazy writer never even created them. The file count will always be 0 and the folder will never exist. I can change the test to ensure that but it won't really check any of `abort`'s behaviour, will it? -- 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]
