chamikaramj commented on code in PR #29859:
URL: https://github.com/apache/beam/pull/29859#discussion_r1440956794


##########
runners/core-construction-java/src/main/java/org/apache/beam/runners/core/construction/CoderTranslation.java:
##########
@@ -60,6 +62,9 @@ private static class DefaultTranslationContext implements 
TranslationContext {}
   // TODO: standardize such things
   public static final String JAVA_SERIALIZED_CODER_URN = 
"beam:coders:javasdk:0.1";
 
+  public static final List<String> KNOWN_EXTERNAL_CODERS =

Review Comment:
   I meant any coder proto that would have hit "Unknown Coder URN %s. Known 
URNs: %s" here. I did not want to override the current behavior (though I'm not 
sure if we actually hit this in practice).
   
   Apart from that, if we want to override the behavior for all known coders I 
can go ahead and do that.



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

Reply via email to