XComp commented on code in PR #21252:
URL: https://github.com/apache/flink/pull/21252#discussion_r1027592579


##########
flink-connectors/flink-connector-pulsar/src/test/java/org/apache/flink/connector/pulsar/testutils/runtime/PulsarRuntimeOperator.java:
##########
@@ -180,6 +180,10 @@ public void createTopic(String topic, int 
numberOfPartitions) {
         }
     }
 
+    public void createSchema(String topic, Schema<?> schema) {
+        sneakyAdmin(() -> admin().schemas().createSchema(topic, 
schema.getSchemaInfo()));

Review Comment:
   Thanks for the brief clarification, @syhily . I created FLINK-30109 to cover 
the issue. We either should add more documentation clarifying why we do this or 
removing it as it's a workaround that might raise questions to readers. Let's 
move the discussion into FLINK-30109. I'm curious about your view on that and 
why the Pulsar API is forcing us to do this exception transformation.



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