Skip to content

Last Step: Remove Lombok from milvus sdk-core#1615

Merged
sre-ci-robot merged 10 commits intomilvus-io:master_remove_lombokfrom
yeyinglang:master_remove_lombok
Sep 22, 2025
Merged

Last Step: Remove Lombok from milvus sdk-core#1615
sre-ci-robot merged 10 commits intomilvus-io:master_remove_lombokfrom
yeyinglang:master_remove_lombok

Conversation

@yeyinglang
Copy link
Copy Markdown
Contributor

issue1469

tips: compatible with the previous version

Remove Lombok from milvus sdk-core
image

for some reasons , we don't allow to install docker, so some unit tests can't run successfully locally
image

lentitude2tk and others added 9 commits September 20, 2025 21:03
Signed-off-by: lentitude2tk <xushuang.hu@zilliz.com>
Signed-off-by: yhmo <yihua.mo@zilliz.com>
Signed-off-by: yhmo <yihua.mo@zilliz.com>
Signed-off-by: lentitude2tk <xushuang.hu@zilliz.com>
Signed-off-by: lentitude2tk <xushuang.hu@zilliz.com>
Signed-off-by: kun <kunyinglang@163.com>
StructArray: schema support (milvus-io#448)

* struct schema

Signed-off-by: SpadeA <tangchenjie1210@gmail.com>

* make all

Signed-off-by: SpadeA <tangchenjie1210@gmail.com>

* add field id to struct schema

Signed-off-by: SpadeA <tangchenjie1210@gmail.com>

* rename StructSchema to StructField

Signed-off-by: SpadeA <tangchenjie1210@gmail.com>

* some progress

Signed-off-by: SpadeA <tangchenjie1210@gmail.com>

* array of vector

Signed-off-by: SpadeA <tangchenjie1210@gmail.com>

* array of vector

Signed-off-by: SpadeA-Tang <tangchenjie1210@gmail.com>

* struct to array struct

Signed-off-by: SpadeA <tangchenjie1210@gmail.com>

* refine name

Signed-off-by: SpadeA <tangchenjie1210@gmail.com>

* rename

Signed-off-by: SpadeA <tangchenjie1210@gmail.com>

* refine position

Signed-off-by: SpadeA <tangchenjie1210@gmail.com>

* rename

Signed-off-by: SpadeA <tangchenjie1210@gmail.com>

* rename

Signed-off-by: SpadeA <tangchenjie1210@gmail.com>

* add type params

Signed-off-by: SpadeA <tangchenjie1210@gmail.com>

* rename

Signed-off-by: SpadeA <tangchenjie1210@gmail.com>

* address comment

Signed-off-by: SpadeA <tangchenjie1210@gmail.com>

* address comment

Signed-off-by: SpadeA <tangchenjie1210@gmail.com>

---------

Signed-off-by: SpadeA <tangchenjie1210@gmail.com>
Signed-off-by: SpadeA-Tang <tangchenjie1210@gmail.com>
@yeyinglang yeyinglang force-pushed the master_remove_lombok branch 2 times, most recently from cec8803 to 3102f6f Compare September 20, 2025 14:17
Signed-off-by: kun <kunyinglang@163.com>
@yhmo
Copy link
Copy Markdown
Contributor

yhmo commented Sep 22, 2025

/lgtm
/approve

@sre-ci-robot
Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: yeyinglang, 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 added the ci-passed label Sep 22, 2025
@sre-ci-robot sre-ci-robot merged commit 4312f0f into milvus-io:master_remove_lombok Sep 22, 2025
5 of 6 checks passed
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.

4 participants