iemejia 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_r386080476
##########
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:
Interesting. I have not thought about making KafkaRecord 'schema' like good
point. There are some consequences on that that are still not clear to me (like
how will we deal with the runtime resolution part of Schemas for KV that we do
now with the Confluent Schema Registry support. I am going to give it a try and
ping you once I have something in the other PR #10978. Let's continue that
discussion there.
----------------------------------------------------------------
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]
With regards,
Apache Git Services
