[AVRO-3039] ClassCache: Cached class map key is too broad

• Type: Bug
• Status: Open
• Priority: Major
• Resolution: Unresolved
• Affects Version/s: 1.10.1
• Component/s: csharp
• Assignee: Unassigned
• Reporter: John Gonyo
• Created: 11/Feb/21 16:04
• Updated: 18/Mar/22 20:43

Description

Currently ClassMap's cache is keyed by the schema full name.  This restricts clients to providing one Plain Old C# Object (POCO) model per schema.  In some cases a client might wish to provide multiple POCO per schema name, perhaps to support multiple versions of the same schema name in the same runtime.

A fix is to use the full RecordSchema as the key instead of just the name.

Additionally, there's a bug in ReflectDefaultReader that can access the incorrect class member property.  Only the reader schema should be used to determine the correct DotNetClass for reading.