SEZ9 commented on issue #12008: URL: https://github.com/apache/seatunnel/issues/12008#issuecomment-5476206192
Confirmed valid bug: the schema converter preserves field name casing while RowToAvroConverter lowercases before GenericRecordBuilder.set(...), causing a case-sensitive lookup miss (NPE on field.pos()). The fix should use the original field name in RowToAvroConverter, including the nested ROW path. Since you already have failing regression tests in AvroConverterTest for top-level and nested mixed-case fields, would you be willing to open a PR with the fix plus those tests? That would speed up review. Note this affects the shared avro format module, so connectors using it (e.g., Kafka, Pulsar) are impacted; please verify the deserialization path (AvroToRowConverter) does not have a matching lowercase assumption that would need updating for consistency. <!-- streview-comment:693 --> -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
