kbendick commented on pull request #2948: URL: https://github.com/apache/iceberg/pull/2948#issuecomment-895834862
> Looks good to me overall. I'd like to get this into 0.12.0 to fix the failure, too. > > Out of curiosity, how was this passing before? Wouldn't this have failed before the assert fix? To answer your question somewhat more directly @rdblue of how this was passing before, this doesn't appear to be run during any of our existing automation. Even running them manually, one has to have the environment set up for it. TIL. -- 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]
