[BEAM-12464] Change ProtoSchemaTranslator beam schema creation to match the order for protobufs containing Oneof fields

• Type: Improvement
• Status: Resolved
• Priority: P3
• Resolution: Fixed
• Fix Version/s: 2.37.0
• Component/s: extensions-java-protobuf
• Assignee: Reuben van Ammers
• Reporter: Reuben van Ammers
• Created: 09/Jun/21 01:04
• Updated: 13/Jan/22 19:13
• Resolved: 13/Jan/22 19:13

Description

Currently, when ProtoSchemaTranslator creates the beam schema from a protobuf definition it always puts the Oneofs at the start of the beam schema due to Oneofs being created from the code first. This means that the order of the fields doesn't match the order of the protobuf defintion. As the schema generation is used when converting from beam rows to protobufs, it additionally means that it is impossible to convert from a beam row where the oneof fields are not the first fields in the beamrow.

Issue Links

• links to: GitHub Pull Request #14974