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_r385169295
 
 

 ##########
 File path: 
sdks/java/core/src/main/java/org/apache/beam/sdk/schemas/SchemaCoder.java
 ##########
 @@ -59,7 +60,7 @@
 @Experimental(Kind.SCHEMAS)
 public class SchemaCoder<T> extends CustomCoder<T> {
   // This contains a map of primitive types to their coders.
-  public static final ImmutableMap<TypeName, Coder> CODER_MAP =
+  public static final Map<TypeName, Coder> CODER_MAP =
 
 Review comment:
   This change is required because we should not be exposing Guava (even our 
own vendored version) in the public API and it was surfaced by 
`GcpCoreApiSurfaceTest` break

----------------------------------------------------------------
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

Reply via email to