Skip to content

Delete equals() and hashCode() methods, rename Builder, reformat code#1654

Merged
sre-ci-robot merged 1 commit intomilvus-io:masterfrom
yhmo:ma
Oct 29, 2025
Merged

Delete equals() and hashCode() methods, rename Builder, reformat code#1654
sre-ci-robot merged 1 commit intomilvus-io:masterfrom
yhmo:ma

Conversation

@yhmo
Copy link
Copy Markdown
Contributor

@yhmo yhmo commented Oct 28, 2025

The reason we renamed the Bilder class name from "Builder" to "XXXBuilder" is: the lombok @Builder and @SuperBuilder generate Builder class with name "XXXBuilder", user might explicitly declare a build like this:

XXX.XXXBuilder builder = XXX.builder();

To ensure compatibility, we rename Builder class name to "XXXBuilder"

#1651

@sre-ci-robot
Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: yhmo

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@yhmo yhmo force-pushed the ma branch 3 times, most recently from bb1b62a to 1dca659 Compare October 28, 2025 11:09
@yhmo yhmo changed the title Delete equals() and hashCode() methods Delete equals() and hashCode() methods, reformat code Oct 28, 2025
@yhmo yhmo force-pushed the ma branch 3 times, most recently from 7368bf1 to 98a0c89 Compare October 29, 2025 04:40
@mergify mergify Bot removed the ci-passed label Oct 29, 2025
@mergify mergify Bot added the ci-passed label Oct 29, 2025
@yhmo yhmo force-pushed the ma branch 5 times, most recently from c76e3c9 to 60608c9 Compare October 29, 2025 09:49
Signed-off-by: yhmo <yihua.mo@zilliz.com>
@yhmo yhmo changed the title Delete equals() and hashCode() methods, reformat code Delete equals() and hashCode() methods, rename Builder, reformat code Oct 29, 2025
@yhmo yhmo added the lgtm label Oct 29, 2025
@sre-ci-robot sre-ci-robot merged commit 1744551 into milvus-io:master Oct 29, 2025
5 checks passed
@yhmo yhmo deleted the ma branch October 30, 2025 11:14
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.

2 participants