[
https://issues.apache.org/jira/browse/AVRO-2131?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16703706#comment-16703706
]
ASF subversion and git services commented on AVRO-2131:
-------------------------------------------------------
Commit 0beb39d78f6bdccb803b8c6eb58af02b2cc80429 in avro's branch
refs/heads/master from [~jeff.maxwell]
[ https://gitbox.apache.org/repos/asf?p=avro.git;h=0beb39d ]
AVRO-2131 Modified Schema.parse(JsonNode schema, Names names) to allow Unions
with self references. (#278)
> Records with unions with self references fail to parse.
> -------------------------------------------------------
>
> Key: AVRO-2131
> URL: https://issues.apache.org/jira/browse/AVRO-2131
> Project: Apache Avro
> Issue Type: Bug
> Components: java
> Affects Versions: 1.8.1
> Environment: Java 1.8.0_152-b1
> Reporter: Jeff Maxwell
> Priority: Major
>
> Records with unions with self references fail to parse.
> The example below fails to parse with {{"Type not supported: Node"}}
> {code:javascript}
> [
> {
> "namespace": "tree",
> "type": "record",
> "name": "Node",
> "fields": [
> {
> "name": "left",
> "type": [
> "null",
> {
> "type": "Node"
> }
> ],
> "default": null
> },
> {
> "name": "right",
> "type": [
> "null",
> {
> "type": "Node"
> }
> ],
> "default": null
> }
> ]
> }
> ]
> {code}
> When we don't allow nullability it parses successfully:
> {code:javascript}
> [
> {
> "namespace": "tree",
> "type": "record",
> "name": "Node",
> "fields": [
> {
> "name": "left",
> "type": "Node"
> },
> {
> "name": "right",
> "type": "Node"
> }
> ]
> }
> ]
> {code}
> The root cause: When the second element of the union, {{\{"type":"Node"}\}},
> is parsed there is no path that can successfully handle the {{JsonNode}}.
> The solution is to add this logic to the {{Schema.parse(JsonNode schema,
> Names names)}} method:
> {code:java}
> } else { //For unions with self reference
> Name nameFromType = new Name(type, names.space);
> if (names.containsKey(nameFromType)) {
> return names.get(nameFromType);
> }
> throw new SchemaParseException("Type not supported: "+type);
> }
> {code}
> Pull request attached
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
