ldadima commented on PR #27508: URL: https://github.com/apache/flink/pull/27508#issuecomment-3931019990
> Hey @ldadima, suggestion: take a look at `RowDataSerializer.toBinaryRow().copy()` to workaround the issue you mentioned. It's a relatively common practice used in some places, [see example](https://github.com/apache/flink/blob/94b5f031a785d16077d870fe9e009d168077430b/flink-table/flink-table-runtime/src/main/java/org/apache/flink/table/runtime/operators/sort/StreamSortOperator.java#L99). Hey @gustavodemorais, As I recall, we wanted to avoid creating RowDataSerializers repeatedly, but now, with this solution, we'll be creating RowData each time (as I wrote above generic row data, writing fields to binary row data, copying binary row data). Isn't that a problem? What do you think of the solution already pushed? -- 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]
