[GitHub] [pulsar-client-cpp] BewareMyPower commented on pull request #142: [feat] Support AUTO_PUBLISH schema.

[GitBox <gi...@apache.org>]

BewareMyPower commented on PR #142:
URL: https://github.com/apache/pulsar-client-cpp/pull/142#issuecomment-1354186304

   It looks like this PR mainly solves the issue that the producer cannot be created when `isSchemaValidationEnforced` is true. But the validation still requires the new dependencies like Avro. Otherwise, the `AUTO_PUBLISH` schema with no validation could be weird.
   
   We should not introduce the validation logic into the C++ client. Maybe we can think of a way to support passing a customized message builder so that we can perform the validation on a generic type without exposing the internal code.
   
   > If a schema exists in a topic, but the producer does not know it. He still wanted to send data to the topic. (The scenario in which the DLQ sends a message
   
   Could you describe in which case could the DLQ topic have schema? I thought he topic could only be created by the client itself, right?


-- 
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: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org