[FLINK-39672][docs] Document Java records as supported POJO types#28179
Conversation
snuyanzin
left a comment
There was a problem hiding this comment.
@liuml07 instead of having AI as a co-author can you changed it to generated by as mentioned at
https://github.com/apache/flink/blame/1e6a73616eb4e224a149e684c832f20bcba93148/AGENTS.md#L327
0c6193c to
37bfe2a
Compare
|
Thanks @snuyanzin I have addressed that. A quick question is in old branches which are still active (<=release-2.2) There is no AGENTS.md file there. Isn't it better to have that as well so when working with backports, agents can find and honor the same AGENTS.md |
yes makes sense to backport to branches |
|
@snuyanzin Thanks for the reply. I have filed pull requests for the new backport JIRA FLINK-39714 Each PR has two commits: one faithful copy from master branch, and the other adapt for the release branch (e.g. update modules / files mentioned that are non-existent) |
What is the purpose of the change
This backports PR #28149 to
release-1.20for FLINK-39672Adapted for release-1.20: drop the older one-liner about Java records introduced in 10157ff, which is now superseded by the more detailed POJO-types description added by this commit.
Java records have been supported as POJO types since Flink 1.19 (FLINK-32380), but the only doc mention lived on the release-1.20 java_compatibility page and was inadvertently removed on master by FLINK-37339.
Brief change log
Parallel updates in docs/content.zh.
Verifying this change
Doc-only change.
Does this pull request potentially affect one of the following parts:
@Public(Evolving): (yes / no)Documentation
Was generative AI tooling used to co-author this PR?
Generated-by: Claude Opus 4.7