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

[BP-1.14][hotfix][table-runtime] Fix ArrayDataSerializer null fields writing #19254

Merged
merged 1 commit into from
Jun 10, 2022

Conversation

snuyanzin
Copy link
Contributor

Unchanged backport of 0046a1c on release-1.14

@snuyanzin snuyanzin changed the title [BP 1.14][hotfix][table-runtime] Fix ArrayDataSerializer null fields writing [BP-1.14][hotfix][table-runtime] Fix ArrayDataSerializer null fields writing Mar 28, 2022
@flinkbot
Copy link
Collaborator

flinkbot commented Mar 28, 2022

CI report:

Bot commands The @flinkbot bot supports the following commands:
  • @flinkbot run azure re-run the last Azure build

@snuyanzin
Copy link
Contributor Author

@knaufk since it was found during knaufk/flink-faker#28
could you please have a look once you have time?

@knaufk
Copy link
Contributor

knaufk commented Jun 9, 2022

@snuyanzin I am not familiar with this part of the code base. @wuchong could you have a quick look?

@wuchong
Copy link
Member

wuchong commented Jun 10, 2022

@snuyanzin could you rebase the branch to the latest realese-1.14 and trigger the CI again?

@snuyanzin
Copy link
Contributor Author

@flinkbot run azure

@snuyanzin
Copy link
Contributor Author

@snuyanzin could you rebase the branch to the latest realese-1.14 and trigger the CI again?

@wuchong done

Copy link
Member

@wuchong wuchong left a comment

Choose a reason for hiding this comment

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

LGTM. I will merge it.

@wuchong wuchong merged commit 0c061da into apache:release-1.14 Jun 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants