poorbarcode commented on code in PR #20888:
URL: https://github.com/apache/pulsar/pull/20888#discussion_r1275742841
##########
pulsar-broker/src/test/java/org/apache/pulsar/schema/SchemaTest.java:
##########
@@ -1290,6 +1290,33 @@ private void testIncompatibleSchema() throws Exception {
assertThrows(SchemaSerializationException.class, message2::getValue);
}
+ /**
+ * This test just ensure that schema check still keeps the original logic:
if there has any producer, but no schema
+ * was registered, the new consumer could not register new schema.
+ * TODO: I think this design should be improved: if a producer used
"AUTO_PRODUCE_BYTES" schema, we should allow
+ * the new consumer to register new schema. But before we can solve
this problem, we need to modify
+ * "CmdProducer" to let the Broker know that the Producer uses a
schema of type "AUTO_PRODUCE_BYTES".
Review Comment:
Done, it is https://github.com/apache/pulsar/issues/20891
--
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]
