ldadima commented on PR #27508: URL: https://github.com/apache/flink/pull/27508#issuecomment-4037023516
Hi @gustavodemorais Code changed according your comments About suggestions: 1. Great! I'll create a ticket 2. This also leads to problems with backward compatibility. Also for nexmark Q20 gives a performance degradation of about 7%. Also, I don't consider it a problem that an extra rule will be added, it will have little impact. 3. I agree. Need tech discussion about it. Can you please clarify? Did I understand correctly that you agree with creating tickets for issues 1 (Generic/Binary row data) and 2 (binary multi join)? But for issue 3 (filter pushdown), additional discussion is needed and we won't create a ticket for now? -- 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]
