reuvenlax commented on a change in pull request #10974: [BEAM-9384] Add
SchemaRegistry.getSchemaCoder to get SchemaCoders for registered types
URL: https://github.com/apache/beam/pull/10974#discussion_r386038872
##########
File path:
sdks/java/core/src/main/java/org/apache/beam/sdk/schemas/SchemaRegistry.java
##########
@@ -288,6 +281,38 @@ public void registerSchemaProvider(SchemaProvider
schemaProvider) {
return getProviderResult((SchemaProvider p) ->
p.fromRowFunction(typeDescriptor));
}
+ /**
+ * Retrieve a {@link SchemaCoder} for a given {@link Class} type. If no
schema exists, throws
+ * {@link * NoSuchSchemaException}.
+ */
+ public <T> SchemaCoder<T> getSchemaCoder(Class<T> clazz) throws
NoSuchSchemaException {
+ return getSchemaCoder(TypeDescriptor.of(clazz));
+ }
+
+ /**
+ * Retrieve a {@link SchemaCoder} for a given {@link TypeDescriptor} type.
If no schema exists,
+ * throws {@link * NoSuchSchemaException}.
+ */
+ public <T> SchemaCoder<T> getSchemaCoder(TypeDescriptor<T> typeDescriptor)
+ throws NoSuchSchemaException {
+ return SchemaCoder.of(
+ getSchema(typeDescriptor),
+ typeDescriptor,
+ getToRowFunction(typeDescriptor),
+ getFromRowFunction(typeDescriptor));
Review comment:
It's useful when integrating schema code with code that does not yet
understand schemas.
In the KafkaIO example I think that the ideal solution would be to allow a
Schema on KafkaRecord (this probably requires us to add Java generic type
awareness to schema inference though), in which case they keyCoder and
valueCoder isn't needed. I agree that allowing easy inference of SchemaCoder
allows for lower-effort integration of schemas in code like this, though
hopefully this is just a short-term solution.
----------------------------------------------------------------
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:
us...@infra.apache.org
With regards,
Apache Git Services
