Improve test coverage for Thrift legacy schemas

Description

There are a few corner cases in schema metadata parsing, for schemas that were migrated from Thrift (see ).

We should cover those in unit or integration tests. The tests should be realistic but performing a Thrift migration during the tests would be overkill. Maybe a good compromise is to do a Thrift migration manually, then emulate the resulting schema with mocks like in TableParserTest.

Environment

None

Pull Requests

None

Status

Assignee

Unassigned

Reporter

Olivier Michallat

Labels

None

PM Priority

None

Reproduced in

None

External issue ID

None

External issue ID

None

External issue ID

None

External issue ID

None

External issue ID

None

External issue ID

None

Doc Impact

None

Reviewer

None

Size

None

Fix versions

Priority

Minor
Configure