TheNeuralBit commented on a change in pull request #12376:
URL: https://github.com/apache/beam/pull/12376#discussion_r461701010
##########
File path:
sdks/java/core/src/main/java/org/apache/beam/sdk/expansion/ExternalTransformRegistrar.java
##########
@@ -30,5 +30,5 @@
public interface ExternalTransformRegistrar {
/** A mapping from URN to an {@link ExternalTransformBuilder} class. */
- Map<String, Class<? extends ExternalTransformBuilder>> knownBuilders();
+ Map<String, Class<? extends ExternalTransformBuilder<?, ?, ?>>>
knownBuilders();
Review comment:
FYI @sclukas77 you may run into some merge conflicts with this PR when
you change this interface to `Map<String, ExternalTransformBuilder>`
----------------------------------------------------------------
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]
