Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Flink: Backport #11662 Fix range distribution npe when value is null to Flink 1.18 and 1.19 #11745

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Guosmilesmile
Copy link
Contributor

This PR backports the Fix range distribution npe when value is null (and corresponding tests).

Refer to pr #11662

@github-actions github-actions bot added the flink label Dec 11, 2024
@Guosmilesmile
Copy link
Contributor Author

@pvary here's the PR to backport for Fix range distribution npe when value is null to Flink v1.18 and v1.19.

Could you please take some time to help review it?

@pvary
Copy link
Contributor

pvary commented Dec 11, 2024

@Guosmilesmile: Was this a clean backport, or you had to modify things to apply the changes?

Copy link

@mxm mxm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the backport. LGTM!

@Guosmilesmile
Copy link
Contributor Author

@pvary Hi Peter,
Is there anything that needs to be modified in this backport? Could you please take some time to review it? Thank you very much!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants