[jira] [Updated] (AVRO-3104) avro-python __eq__ method of PrimitiveSchema, EnumSchema and FixedSchema throws exception

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

[jira] [Updated] (AVRO-3104) avro-python __eq__ method of PrimitiveSchema, EnumSchema and FixedSchema throws exception

Dave Cole (Jira)

     [ https://issues.apache.org/jira/browse/AVRO-3104?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Michael A. Smith updated AVRO-3104:
-----------------------------------
    Fix Version/s: 1.11.0
       Resolution: Fixed
           Status: Resolved  (was: Patch Available)

> avro-python __eq__ method of PrimitiveSchema, EnumSchema and FixedSchema throws exception
> -----------------------------------------------------------------------------------------
>
>                 Key: AVRO-3104
>                 URL: https://issues.apache.org/jira/browse/AVRO-3104
>             Project: Apache Avro
>          Issue Type: Bug
>          Components: python
>    Affects Versions: 1.10.2
>            Reporter: Thomas Larsson
>            Priority: Minor
>             Fix For: 1.11.0
>
>         Attachments: AVRO-3104.patch
>
>
> The __eq__ method of the [PrimitiveSchema|https://github.com/apache/avro/blob/master/lang/py3/avro/schema.py#L629], EnumSchema and FixedSchema assume that the compared object has an attribute called "props" which makes a statement like this (for example) throw an exception:
> {code:python}
> o = PrimitiveSchema(...)
> if o == "null":
>   print("Not same")
> {code}
>    
> The exception message looks like:
> {code:bash}
> AttributeError: 'str' object has no attribute 'props'
> {code}
> A more robust equals method would start by ensuring type equality before doing other comparisons. Alternatively (though less pretty) the way the other Schema types implement it could be used, for example [ArraySchema|https://github.com/apache/avro/blob/master/lang/py3/avro/schema.py#L772]



--
This message was sent by Atlassian Jira
(v8.3.4#803005)