[GitHub] [pulsar] hdiode-florian opened a new issue, #16457: Invalid use of `type`s in error messages in `definition.py`

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

hdiode-florian opened a new issue, #16457:
URL: https://github.com/apache/pulsar/issues/16457

   **Describe the bug**
   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, for example:
   https://github.com/apache/pulsar/blob/66e8206f129a7ea303099af1317f78652240f985/pulsar-client-cpp/python/pulsar/schema/definition.py#L29
   
   **To Reproduce**
   ```python
   from pulsar.schema import Array, Integer
   f = Array(Integer)
   ```
   
   ```
   TypeError: can only concatenate str (not "type") to str
   ```
   
   **Expected behavior**
   ```
   Exception: Argument Integer is not a Record or a Field
   ```
   
   **Additional context**
   The same mistake was made in several places in that file.
   


-- 
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] github-actions[bot] commented on issue #16457: Invalid use of `type`s in error messages in `definition.py`

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

github-actions[bot] commented on issue #16457:
URL: https://github.com/apache/pulsar/issues/16457#issuecomment-1207316081

   The issue had no activity for 30 days, mark with Stale label.


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