hulincup commented on PR #28931: URL: https://github.com/apache/flink/pull/28931#issuecomment-5210617394
Hi @SEPURI-SAI-KRISHNA, you're right, and I apologize for missing your claim on the JIRA ticket before opening this PR. That was my mistake — I should have scanned the comments for assignment signals first. Since @raminqaf has started reviewing here and you've generously offered to hand over the test cases, I've incorporated your field-reference tests directly (credited in the new commit). The codegen path is now covered via `f4`/`f5`/`f6` (TINYINT/SMALLINT/BIGINT), with one constant case kept for the reducer path. Thanks for the verified patch and the detailed review. -- 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]
