Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

enhance: Sync v1.0.x pr to main branch (part4) #357

Merged
merged 13 commits into from
Feb 19, 2025

Conversation

congqixia
Copy link
Collaborator

congqixia and others added 12 commits February 19, 2025 19:07
Signed-off-by: Congqi Xia <[email protected]>
This PR set etcd address to default 2379 port if user param does not
contain port.

Signed-off-by: Congqi Xia <[email protected]>
Add `scan-deltalog` command to scan deltalogs and locate specified
existence

---------

Signed-off-by: Congqi Xia <[email protected]>
The basePath & metaPath shall be separated since most of meta info are
stored under metaPath but some other info (say config-etcd) are stored
under basePath

Signed-off-by: Congqi Xia <[email protected]>
Add balance-segment command to manual load balance segments

Signed-off-by: Congqi Xia <[email protected]>
Signed-off-by: Congqi Xia <[email protected]>
Previously, scan-binlog command need to manually provide outputFields
for pk column even the pk column & ts column is always fetched.

This PR unifies `scan-binlog` & `scan-deltalog` command to have the
default `$pk` & `$timestamp` env value for filter expression.

Signed-off-by: Congqi Xia <[email protected]>
Signed-off-by: bigsheeper <[email protected]>
Signed-off-by: Congqi Xia <[email protected]>
Previously common.ListSegmentIndex has default timeout of 3 seconds,
which could cause command fail when segment index number is large

Signed-off-by: Congqi Xia <[email protected]>
@sre-ci-robot
Copy link
Collaborator

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: congqixia

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

The pull request process is described here

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

Copy link

mergify bot commented Feb 19, 2025

@congqixia Thanks for your contribution. Please submit with DCO, see the contributing guide https://github.com/milvus-io/milvus/blob/master/CONTRIBUTING.md#developer-certificate-of-origin-dco.

Signed-off-by: Congqi Xia <[email protected]>
@weiliu1031
Copy link
Collaborator

/lgtm

@sre-ci-robot sre-ci-robot added the lgtm look good to me label Feb 19, 2025
@sre-ci-robot sre-ci-robot merged commit f41743b into milvus-io:main Feb 19, 2025
4 checks passed
@congqixia congqixia deleted the enhance/sync_main_part4 branch February 20, 2025 07:20
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