RussellSpitzer commented on pull request #1264: URL: https://github.com/apache/iceberg/pull/1264#issuecomment-670064241
We've been talking about the testing issue for a while, one of the biggest issues is that we cannot use the same code from TestRemoveSnapshots because it all relies on tons of inner class parameters and fields so it's extremely difficult (at least from my perspective) to setup the cherry pick cases. Even if we wanted to use the base class from spark-core for testing, we would then be unable to inherit from the Spark Base Action test class. I'll take a look at getting in alternative versions of these tests ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
