[AVRO-2466] Fix a malformed schema in the share/test/schemas directory

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 1.9.2
• Component/s: misc
• Assignee: Kengo Seki
• Reporter: Kengo Seki
• Created: 06/Jul/19 13:53
• Updated: 07/Feb/20 08:46
• Resolved: 23/Jul/19 09:47

Description

The schema defined in share/test/schemas/reserved.avsc is invalid because of its trailing comma:

$ python -c 'from avro.schema import parse; parse(open("../../share/test/schemas/reserved.avsc").read())'
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/home/sekikn/repo/avro/lang/py/src/avro/schema.py", line 976, in parse
    json_data = json.loads(json_string)
  File "/usr/lib/python2.7/json/__init__.py", line 339, in loads
    return _default_decoder.decode(s)
  File "/usr/lib/python2.7/json/decoder.py", line 367, in decode
    raise ValueError(errmsg("Extra data", s, end, len(s)))
avro.schema.SchemaParseException: Error parsing JSON: {"name": "org.apache.avro.test.Reserved", "type": "enum",
 "symbols": ["default","class","int"]},
, error = Extra data: line 2 column 39 - line 3 column 1 (char 96 - 98)
$ cat ../../share/test/schemas/reserved.avsc
{"name": "org.apache.avro.test.Reserved", "type": "enum",
 "symbols": ["default","class","int"]},

This file doesn't seem to be used as far as I investigated, but I'd rather fix this than remove since it might be useful for some test.

Issue Links

• links to: GitHub Pull Request #578