SEZ9 commented on issue #12657: URL: https://github.com/apache/seatunnel/issues/12657#issuecomment-6029738624
Thanks for the detailed report and for linking the unmerged #10194 and its review note about the nested MAP descriptor problem. Since you are willing to submit a PR, please include the three cases described here as regression tests: the exact `C_STRING` write, the `c_string`/`C_String` case-variant read, and the `address.attributes` MAP inside a nested ROW, so the owning-message lookup fix is covered. Please also keep the lookup behavior symmetric between RowToProtobufConverter and ProtobufToRowConverter, with exact-name matching tried first and the case-insensitive fallback retained so existing schemas keep working. Could you paste the actual exception/stack trace from the failing exact-case write on the dev commit you mentioned? That will help reviewers confirm the current behavior quickly. <!-- streview-comment:1571 --> -- 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]
