nodece commented on a change in pull request #13983:
URL: https://github.com/apache/pulsar/pull/13983#discussion_r794316399
##########
File path: conf/broker.conf
##########
@@ -1283,16 +1284,12 @@ splitTopicAndPartitionLabelInPrometheus=false
# The schema storage implementation used by this broker
schemaRegistryStorageClassName=org.apache.pulsar.broker.service.schema.BookkeeperSchemaStorageFactory
-# Enforce schema validation on following cases:
-#
-# - if a producer without a schema attempts to produce to a topic with schema,
the producer will be
-# failed to connect. PLEASE be carefully on using this, since non-java
clients don't support schema.
-# if you enable this setting, it will cause non-java clients failed to
produce.
+# Whether to enable schema validation.
+# When enabled, if a producer without a schema attempts to produce to a topic
with schema, the producer will be rejected and disconnected.
Review comment:
> 1. or "produce to a topic with schemas"?
"The producer sends a message without schema to the topic that has one or
many schemas"?
> 2. Write in the simple present tense as much as possible if you are
covering facts that were, are, and forever shall be true. Details:
https://docs.google.com/document/d/1lc5j4RtuLIzlEYCBo97AC8-U_3Erzs_lxpkDuseU0n4/edit#bookmark=id.e8uqh1awkcnp
Thanks
--
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]
