ldadima commented on PR #27508: URL: https://github.com/apache/flink/pull/27508#issuecomment-3936373175
> 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? Ok, I understand. It's partially the same I will rerun CI. I hope, it will success -- 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]
