scwhittle commented on pull request #15416: URL: https://github.com/apache/beam/pull/15416#issuecomment-950935938
R: @chamikaramj PTAL, I believe this addresses the test failures. The test failures were arising due to the test not specifying a temporary directory for output. The default behavior in this case is to use the output directory as the temporary directory. This didn't work with the previous cl because the temporary files themselves showed up in the output directory then. I modified the PR so that we continue to use a subdirectory for temporary files when we were before, what changes is if that subdirectory is unique for the execution or if it is consistent across executions. -- 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]
