hdiode-florian opened a new pull request, #16459:
URL: https://github.com/apache/pulsar/pull/16459

   Fixes #16457
   
   ### Motivation
   In several places 
https://github.com/apache/pulsar/blob/master/pulsar-client-cpp/python/pulsar/schema/definition.py
 uses incorrect type (name) to string conversions when exceptions are raised 
(see for #16443 for examples). This might not be obvious at first sight because 
the call trace actually contains that somewhat resembles the intended exception 
message. The error occurs when the messages are created by concatenating 
"strings" with the + operator. This fails when one operand is a `type`.
   
   ### Modifications
   Instead of attempting to directly convert a `type` to a `str`, I replaced 
all occurrences with `some_type.__name__`.
   
   ### Verifying this change
   This change is a trivial rework / code cleanup without any test coverage.


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