[GitHub] [pulsar-client-go] zzzming opened a new issue, #793: NewAvroSchema's error lead to panic

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

zzzming opened a new issue, #793:
URL: https://github.com/apache/pulsar-client-go/issues/793

   #### Expected behavior
   
   NewAvroSchema or NewJSONSchema should not panic a client process. If there is an error, it should return error and allow user to deal with that in the client code.
   
   #### Actual behavior
   
   User should handle any codec error if the NewAvroSchema returns an error reference, although a user can still write a panic recovery procedure but that is not a standard way to do error handling.
   
   This is a recoverable error. So the library should not panic.
   
   


-- 
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.apache.org

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

[GitHub] [pulsar-client-go] merlimat closed issue #793: NewAvroSchema's error lead to panic

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

merlimat closed issue #793: NewAvroSchema's error lead to panic
URL: https://github.com/apache/pulsar-client-go/issues/793


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