[
https://issues.apache.org/jira/browse/AVRO-1340?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15691842#comment-15691842
]
ASF GitHub Bot commented on AVRO-1340:
--------------------------------------
GitHub user ksenji opened a pull request:
https://github.com/apache/avro/pull/163
AVRO-1340: use default to allow old readers to specify default enum v…
…alue when encountering new enum symbols
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/ksenji/avro master
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/avro/pull/163.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #163
----
commit f7c775b347150e36f3200516dbce9e7e049506eb
Author: ksenji <[email protected]>
Date: 2016-11-24T00:56:26Z
AVRO-1340: use default to allow old readers to specify default enum value
when encountering new enum symbols
----
> use default to allow old readers to specify default enum value when
> encountering new enum symbols
> -------------------------------------------------------------------------------------------------
>
> Key: AVRO-1340
> URL: https://issues.apache.org/jira/browse/AVRO-1340
> Project: Avro
> Issue Type: Improvement
> Components: spec
> Environment: N/A
> Reporter: Jim Donofrio
> Priority: Minor
>
> The schema resolution page says:
> > if both are enums:
> > if the writer's symbol is not present in the reader's enum, then an
> error is signalled.
> This makes it difficult to use enum's because you can never add a enum value
> and keep old reader's compatible. Why not use the default option to refer to
> one of enum values so that when a old reader encounters a enum ordinal it
> does not recognize, it can default to the optional schema provided one. If
> the old schema does not provide a default then the older reader can continue
> to fail as it does today.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
