hutiefang76 commented on PR #12900: URL: https://github.com/apache/seatunnel/pull/12900#issuecomment-6291777373
Added the same-name case in `7c2cdf5a8d`: `left_parent.detail` contains a STRING field and `right_parent.detail` an INT field. Both present and null values round-trip through the public serializer/deserializer in nullable mode. I also added the option class Javadoc. All 20 Avro tests passed on the current dev integration, including those two cases and the existing default-layout guards; root formatting and verify passed too. The earlier Kafka/Pulsar broker results remain in the description, clearly separated from this test/Javadoc-only follow-up. I agree the nested schema rebuild is worth a separate optimization. It already occurs on the legacy path, and this patch leaves it unchanged; reusing schemas there needs to preserve the legacy nested record names as well. I have kept that out of the nullable-layout fix. The branch is synced with dev `1ce2d78156`, Actions is restored, and remote CI is still pending. -- 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]
