AtharvUrunkar opened a new pull request, #39363: URL: https://github.com/apache/beam/pull/39363
Fixes #39336 ## What is the issue? Artifact staging can fail on Windows when an environment ID contains characters that are invalid in Windows filenames, such as `:`. The environment ID was previously embedded directly into the generated staging filename, causing an `InvalidPathException` before the pipeline could run. ## What does this PR do? * Sanitizes Windows-invalid filename characters in environment IDs before using them in staged artifact filenames. * Adds a regression test using an environment ID containing `:` to verify that artifact staging succeeds on Windows. ## Testing * `./gradlew :runners:java-fn-execution:test --tests "org.apache.beam.runners.fnexecution.artifact.ArtifactStagingServiceTest"` * `./gradlew :runners:java-fn-execution:spotlessCheck` * `./gradlew :runners:java-fn-execution:check` All tests and checks pass successfully. -- 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]
