NicoK opened a new pull request #16426:
URL: https://github.com/apache/flink/pull/16426
## What is the purpose of the change
While working with the PojoSerializer a bit more, I noticed a couple of
minor things that are off in the current tests:
- the test Pojo does not take dumm5 into account for hashCode and equals
- error messages are not so nice (and mix up the order of expected and
actual values)
## Brief change log
- hotfix typo in error message inside `Types.java`
- fix `PojoSerializerTest` user POJO not taking `dumm5` into account for
`hashCode` and `equals`
- nicer error messages in `PojoSerializerTest` via hamcrest
- fix wrong parameter order in `assertSame()`
## Verifying this change
This change is a trivial rework / code cleanup without any new test coverage.
## Does this pull request potentially affect one of the following parts:
- Dependencies (does it add or upgrade a dependency): **no**
- The public API, i.e., is any changed class annotated with
`@Public(Evolving)`: **no**
- The serializers: **no**
- The runtime per-record code paths (performance sensitive): **no**
- Anything that affects deployment or recovery: JobManager (and its
components), Checkpointing, Kubernetes/Yarn, ZooKeeper: **no**
- The S3 file system connector: **no**
## Documentation
- Does this pull request introduce a new feature? **no**
- If yes, how is the feature documented? **not applicable**
--
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: issues-unsubscr...@flink.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
