Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for @AvroAlias annotation #59

Merged
merged 1 commit into from
Mar 20, 2017

Conversation

baharclerode
Copy link
Contributor

To provide a means forward for #58 in 2.9, this implements support for @AvroAlias for records and enums.

@cowtowncoder
Copy link
Member

I guess this makes sense, although I am not really happy about annotation traversal that by-passes AnnotationIntrospector. However given that AnnotationIntrospector method:

public List<PropertyName> findPropertyAliases(Annotated m) ...

suggests access only for properties (although technically it does work for classes too here), and since no access to introspector is available maybe that's ok for now.

@cowtowncoder cowtowncoder merged commit f1ae27d into FasterXML:master Mar 20, 2017
@cowtowncoder cowtowncoder added this to the 2.9.0.pr2 milestone Mar 20, 2017
@cowtowncoder cowtowncoder changed the title [Avro] Add support for @AvroAlias annotation Add support for @AvroAlias annotation Mar 20, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants