gmethvin opened a new pull request #9440:
URL: https://github.com/apache/pulsar/pull/9440
Fixes #9438
### Motivation
Since `MessageId` is an opaque interface that has several subclasses, in
general those subclasses should try to behave the same way with respect to one
another if they refer to the same topic. Also they should follow the standard
contracts with respect to `equals` and `hashCode`.
### Modifications
Provided a single `equals` and `hashCode` implementation for `MessageIdImpl`
that takes into account the batch version as well. Changed `TopicMessageIdImpl`
to not take into account the topic anymore.
### Verifying this change
- Extended unit test for `BatchMessageIdImpl`
- Added unit test for `TopicMessageIdImpl`
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
