wzhero1 opened a new pull request, #791:
URL: https://github.com/apache/flink-agents/pull/791
## What is the purpose of the change
Found during integrate_with_flink doc verification (#743), reported in #773.
The Java DataStream snippet in
`docs/content/docs/development/integrate_with_flink.md`
used `x::getId` — a *bound* method reference on an undefined variable `x`.
Copying
the snippet into a real Java class fails to compile:
```
error: cannot find symbol
symbol: variable x
```
This fixes it by switching to the *unbound* method reference
`YourPojo::getId`,
which matches the function signature of `KeySelector<YourPojo, String>` and
is
consistent with the existing working example in
`docs/content/docs/development/yaml.md`.
## Brief change log
- `integrate_with_flink.md`: `(KeySelector<YourPojo, String>) x::getId`
→ `(KeySelector<YourPojo, String>) YourPojo::getId`
## Verifying this change
This is a documentation-only change. The corrected form is the same unbound
method-reference pattern already used (and verified) in `yaml.md:349`.
## Does this pull request potentially affect one of the following parts:
No runtime/API impact — documentation only.
Closes #773
--
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]
