Skip to content

feat: support equalsAndHashcode for BookieServerInfo#4686

Merged
hangc0276 merged 4 commits into
apache:masterfrom
mattisonchao:feat.equals
Nov 25, 2025
Merged

feat: support equalsAndHashcode for BookieServerInfo#4686
hangc0276 merged 4 commits into
apache:masterfrom
mattisonchao:feat.equals

Conversation

@mattisonchao

@mattisonchao mattisonchao commented Nov 20, 2025

Copy link
Copy Markdown
Member

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
Copy Markdown
Member Author

rerun failure checks

@horizonzy horizonzy left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM.

@mattisonchao

Copy link
Copy Markdown
Member Author

rerun failure checks

2 similar comments
@mattisonchao

Copy link
Copy Markdown
Member Author

rerun failure checks

@mattisonchao

Copy link
Copy Markdown
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 <mattisonchao@gmail.com>

* change order

* fix checkstyle

* fix checkstyle

---------

Signed-off-by: mattisonchao <mattisonchao@gmail.com>
(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 <mattisonchao@gmail.com>

* change order

* fix checkstyle

* fix checkstyle

---------

Signed-off-by: mattisonchao <mattisonchao@gmail.com>
(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 <mattisonchao@gmail.com>

* change order

* fix checkstyle

* fix checkstyle

---------

Signed-off-by: mattisonchao <mattisonchao@gmail.com>
(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