echauchot commented on pull request #14347: URL: https://github.com/apache/beam/pull/14347#issuecomment-825523777
> Thanks for the 2nd round @echauchot ! Working my way through your comments and adding test coverage. Seeing some of behaviour in the tests, I may have to reach out with questions if I can’t get to the bottom of it. > > Somewhat related: can you describe the high-level difference between the IOTest Vs. IOIT files? UTests for IO are not in the proper sense UTests because they depend on external software (embedded in-jvm ES instance provided by elastic test framework). We do so because to be relevant, the tests need to be run against a real backend and not a mock. And we want to fail the build if any regression occurs. ITests are run against elasticsearch instances installed in a cluster in IT environment. The tests there are more aimed for volume (which as proved to be a problem in some cases). -- 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]
