[ 
https://issues.apache.org/jira/browse/AVRO-1147?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13452466#comment-13452466
 ] 

Alexandre Normand commented on AVRO-1147:
-----------------------------------------

It looks good. Indeed, I had missed the @Stringable on map keys. I thought I 
had cleaned up the writeMapKey signature and used Decoder though...but I think 
I may have selected the wrong patch version when uploading. I'll be more 
careful next time. 

I think you can go ahead with the commit. It's going to be very useful. Thanks 
for the quick response!
                
> Allow Stringable Map keys in addition to pure String keys (avro reflection)
> ---------------------------------------------------------------------------
>
>                 Key: AVRO-1147
>                 URL: https://issues.apache.org/jira/browse/AVRO-1147
>             Project: Avro
>          Issue Type: Improvement
>          Components: java
>    Affects Versions: 1.7.0
>            Reporter: Alexandre Normand
>            Assignee: Alexandre Normand
>              Labels: newbie, patch
>         Attachments: AVRO-1147.patch, AVRO-1147.patch
>
>
> I'm using avro reflection for some serialization that doesn't require the 
> best performance for some short-lived data. 
> However, the object I'm trying to serialize uses a {{Map}} with non-String 
> keys (the key class is {{Integer}}) and, because of the strict restriction on 
> Map keys being Strings, it fails. 
> Could we consider allowing Map keys to be *{{Stringable}}* classes (I'm 
> thinking of something similar to Doug's solution in 
> https://issues.apache.org/jira/browse/AVRO-1146)

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to