ViktorCosenza commented on PR #24795: URL: https://github.com/apache/flink/pull/24795#issuecomment-2125705836
I see, Ive got the impression that they were forgotten, not purposely left out because no tests covered writing nested structures ( if there were, the tests would fail and the methods would have been implemented before) Did you got the impression it wasn’t added on purpose? I could add more tests if you think it would help On Wed, 22 May 2024 at 17:26 Jing Ge ***@***.***> wrote: > It looks like those methods were skipped on purpose in #17542 > <https://github.com/apache/flink/pull/17542> and #17542 (comment) > <https://github.com/apache/flink/pull/17542#issuecomment-1954552466> > > — > Reply to this email directly, view it on GitHub > <https://github.com/apache/flink/pull/24795#issuecomment-2125686671>, or > unsubscribe > <https://github.com/notifications/unsubscribe-auth/AHIY6WOWDLNIA3B37L2L6F3ZDT5ODAVCNFSM6AAAAABHZAOTPKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMRVGY4DMNRXGE> > . > You are receiving this because you were mentioned.Message ID: > ***@***.***> > -- 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: issues-unsubscr...@flink.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
