gustavodemorais commented on PR #27508: URL: https://github.com/apache/flink/pull/27508#issuecomment-3935651887
Thanks, @ldadima! > But I'd like to point out that the updated 1st solution doesn't create any serializers at all Yeah, what I meant is that we still do partially the same thing. For example - new BinaryRowData(...) - new BinaryRowWriter(keyRow) -> this does some memory allocation as well The python CI seems to be broken, which seems unrelated. Maybe another rerun and hopefully flaky tests are gone? -- 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]
