search

apache / pulsar-client-python

Apache Pulsar Python client library
https://pulsar.apache.org/
Apache License 2.0
51 stars 39 forks source link

Error message should be f-string, but is just string #144

Closed ben-kearney closed 1 year ago

ben-kearney commented 1 year ago

https://github.com/apache/pulsar-client-python/blob/ba99cd5f372ab645bbb12239dad53668b10075a9/pulsar/schema/schema_avro.py#L93-L94

Since this is a string and not an f-string, we are unable to see which topic+version is having an issue, and we are unable to see the error.

BewareMyPower commented 1 year ago

It looks like a simple fix, would you like to open a PR for it?