[ 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