Skip to content

Conversation

@mattisonchao
Copy link
Member

@mattisonchao mattisonchao commented Nov 20, 2025

Motivation

The BookieServerInfo class is exposed to the metadata store plugin. We need to support equalsAndHashcode to help other plugins encode or decode it with comparison.

FYI: apache/pulsar#25001

Changes

  • Support equalsAndHashcode method for BookieServerInfo and underlying Endpoint

@mattisonchao
Copy link
Member Author

rerun failure checks

Copy link
Member

@horizonzy horizonzy left a comment

Choose a reason for hiding this comment

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

LGTM.

@mattisonchao
Copy link
Member Author

rerun failure checks

2 similar comments
@mattisonchao
Copy link
Member Author

rerun failure checks

@mattisonchao
Copy link
Member Author

rerun failure checks

@zymap zymap closed this Nov 24, 2025
@zymap zymap reopened this Nov 24, 2025
@zymap zymap added this to the 4.18.0 milestone Nov 24, 2025
@hangc0276 hangc0276 merged commit 14c2ffe into apache:master Nov 25, 2025
112 of 181 checks passed
lhotari pushed a commit that referenced this pull request Nov 28, 2025
* feat: support equalsAndHashcode for BookieServerInfo

Signed-off-by: mattisonchao <[email protected]>

* change order

* fix checkstyle

* fix checkstyle

---------

Signed-off-by: mattisonchao <[email protected]>
(cherry picked from commit 14c2ffe)
priyanshu-ctds pushed a commit to datastax/bookkeeper that referenced this pull request Dec 2, 2025
* feat: support equalsAndHashcode for BookieServerInfo

Signed-off-by: mattisonchao <[email protected]>

* change order

* fix checkstyle

* fix checkstyle

---------

Signed-off-by: mattisonchao <[email protected]>
(cherry picked from commit 14c2ffe)
(cherry picked from commit 6338d45)
srinath-ctds pushed a commit to datastax/bookkeeper that referenced this pull request Dec 4, 2025
* feat: support equalsAndHashcode for BookieServerInfo

Signed-off-by: mattisonchao <[email protected]>

* change order

* fix checkstyle

* fix checkstyle

---------

Signed-off-by: mattisonchao <[email protected]>
(cherry picked from commit 14c2ffe)
(cherry picked from commit 6338d45)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants