The provider route doesn't really fit my intention with the file, which is to 
hold multiple PTransformMatchers that may be used by Dataflow but can't go in 
[PTransformMatchers](https://github.com/apache/beam/blob/master/runners/core-construction-java/src/main/java/org/apache/beam/runners/core/construction/PTransformMatchers.java)
 due to not being generic enough. I structured this file similarly to that one 
to allow for supporting additional PTransformMatchers in the future if 
necessary.

Instead how about moving the logic from that static function out of that file 
and into DataflowRunner, and just instantiating the matchers from there.

[ Full content available at: https://github.com/apache/beam/pull/6384 ]
This message was relayed via gitbox.apache.org for [email protected]

Reply via email to