[BEAM-14522] ProtoSchemaTranslator creates NOT NULL beam schema fields even for optional proto fields

• Type: Bug
• Status: Resolved
• Priority: P2
• Resolution: Won't Fix
• Fix Version/s: Not applicable
• Component/s: extensions-java-protobuf
• Assignee: Unassigned
• Reporter: Balázs Németh
• Created: 26/May/22 17:29
• Updated: 27/May/22 02:55
• Resolved: 27/May/22 01:39

Description

ProtoSchemaTranslator is capable of creating Beam schemas (org.apache.beam.sdk.schemas.Schema) from proto classes. The problem is that the code that transforms the non-null org.apache.beam.sdk.schemas.Schema.FieldType into a nullable one is at a wrong place. Instead of doing it for every type for every field that is optional at the proto class after the switch-case it is actually inside the switch-case and only does it for MESSAGE and GROUP types.

Issue Links

• links to: GitHub Pull Request #17764