[GitHub] [beam] kennknowles opened a new issue, #18336: SDK should have runtime hamcrest dependency or should bundle it

[GitBox <gi...@apache.org>]

kennknowles opened a new issue, #18336:
URL: https://github.com/apache/beam/issues/18336

   `TestOptions`, `SerializableMatchers`, and `PAssert` have compile-time dependencies on hamcrest. We scope it `provided` to avoid having it on the classpath at runtime.
   
   This has lots of pain points, but one that recently bit the Spark runner is that when `TestOptions` or any subinterface is registered, we gain a runtime dependency on Hamcest. So users of the Spark runner all had to provide it, even outside of testing.
   
   We should just simplify this to either not have this dependency (perhaps via some hacking to make shading possibly) or declare the dependency.
   
   Imported from Jira [BEAM-2237](https://issues.apache.org/jira/browse/BEAM-2237). Original Jira may contain additional context.
   Reported by: kenn.


-- 
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: github-unsubscribe@beam.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org