[ https://issues.apache.org/jira/browse/AVRO-1667?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15196036#comment-15196036 ]
Zoltan Farkas commented on AVRO-1667: ------------------------------------- The unit test did catch the original issue though, so it is quite useful to have it in. The latest version of the test I complicated it a lot to try to beak my original fix, but I was unable to. It currently contains testing the equivalent of: {code} record Node { array<Node> left; array<Node> right; union {null, Node} center; array<NodePair> nodepairs;; } record NodePair { Node node1; Node node2; } {code} In any case I will proceed with the rebase of AVRO-1723. The unit test for that, does cover this bug as well... thank you! > Parser symbol tree flattening is broken for recursive schemas > ------------------------------------------------------------- > > Key: AVRO-1667 > URL: https://issues.apache.org/jira/browse/AVRO-1667 > Project: Avro > Issue Type: Bug > Affects Versions: 1.7.7 > Reporter: Zoltan Farkas > Assignee: Zoltan Farkas > Fix For: 1.8.1 > > Attachments: AVRO-1667.2.patch, avro-1667.patch > > > Here is a unit test to reproduce: > {noformat} > package org.apache.avro.io.parsing; > import java.io.IOException; > import java.util.HashMap; > import java.util.HashSet; > import java.util.Set; > import junit.framework.Assert; > import org.apache.avro.Schema; > import org.junit.Test; > public class SymbolTest { > private static final String SCHEMA = > "{\"type\":\"record\",\"name\":\"SampleNode\"," > + "\"namespace\":\"org.spf4j.ssdump2.avro\",\n" + > " \"fields\":[\n" + > " {\"name\":\"count\",\"type\":\"int\",\"default\":0},\n" + > " {\"name\":\"subNodes\",\"type\":\n" + > " {\"type\":\"array\",\"items\":{\n" + > " \"type\":\"record\",\"name\":\"SamplePair\",\n" + > " \"fields\":[\n" + > " {\"name\":\"method\",\"type\":\n" + > " {\"type\":\"record\",\"name\":\"Method\",\n" + > " \"fields\":[\n" + > " > {\"name\":\"declaringClass\",\"type\":{\"type\":\"string\",\"avro.java.string\":\"String\"}},\n" > + > " > {\"name\":\"methodName\",\"type\":{\"type\":\"string\",\"avro.java.string\":\"String\"}}\n" > + > " ]}},\n" + > " {\"name\":\"node\",\"type\":\"SampleNode\"}]}}}]}"; > @Test > public void testSomeMethod() throws IOException { > Schema schema = new Schema.Parser().parse(SCHEMA); > Symbol root = Symbol.root(new ResolvingGrammarGenerator() > .generate(schema, schema, new > HashMap<ValidatingGrammarGenerator.LitS, Symbol>())); > validateNonNull(root, new HashSet<Symbol>()); > } > private static void validateNonNull(final Symbol symb, Set<Symbol> seen) { > if (seen.contains(symb)) { > return; > } else { > seen.add(symb); > } > if (symb.production != null) { > for (Symbol s : symb.production) { > if (s == null) { > Assert.fail("invalid parsing tree should not contain > nulls"); > } > if (s.kind != Symbol.Kind.ROOT) { > validateNonNull(s, seen);; > } > } > } > } > } > {noformat} -- This message was sent by Atlassian JIRA (v6.3.4#6332)