damondouglas commented on a change in pull request #17181:
URL: https://github.com/apache/beam/pull/17181#discussion_r834847151
##########
File path:
sdks/java/core/src/main/java/org/apache/beam/sdk/schemas/transforms/TypedSchemaTransformProvider.java
##########
@@ -42,13 +42,13 @@
@Experimental(Kind.SCHEMAS)
public abstract class TypedSchemaTransformProvider<ConfigT> implements
SchemaTransformProvider {
- abstract Class<ConfigT> configurationClass();
+ protected abstract Class<ConfigT> configurationClass();
Review comment:
This resolved the does-not-override method errors when extending this
class.
##########
File path:
sdks/java/core/src/main/java/org/apache/beam/sdk/schemas/transforms/TypedSchemaTransformProvider.java
##########
@@ -42,13 +42,13 @@
@Experimental(Kind.SCHEMAS)
public abstract class TypedSchemaTransformProvider<ConfigT> implements
SchemaTransformProvider {
- abstract Class<ConfigT> configurationClass();
+ protected abstract Class<ConfigT> configurationClass();
/**
* Produce a SchemaTransform from ConfigT. Can throw a {@link
InvalidConfigurationException} or a
* {@link InvalidSchemaException}.
*/
- abstract SchemaTransform from(ConfigT configuration);
+ protected abstract SchemaTransform from(ConfigT configuration);
Review comment:
This resolved the does-not-override method errors when extending this
class.
--
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]
