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

Add database operations to MilvusClient #2152

Merged
merged 1 commit into from
Jul 2, 2024

Conversation

ashkrisk
Copy link
Contributor

Allows the following operations to be performed through the MilvusClient interface:

  • create_database
  • drop_database
  • list_databases

Resolves #2151

@mergify mergify bot added the dco-passed label Jun 26, 2024
@XuanYang-cn XuanYang-cn self-assigned this Jun 27, 2024
@XuanYang-cn
Copy link
Contributor

We'll fix the lint issue

@ashkrisk
Copy link
Contributor Author

ashkrisk commented Jun 28, 2024

Thanks for reviewing, @XuanYang-cn. Added the return type information.

@ashkrisk ashkrisk requested a review from XuanYang-cn June 28, 2024 20:39
@XuanYang-cn
Copy link
Contributor

blocked by #2159

create_database, drop_database and list_databases

Signed-off-by: Ashwin Krishna Kumar <[email protected]>
@ashkrisk
Copy link
Contributor Author

ashkrisk commented Jul 2, 2024

Rebased to include the lint changes

@mergify mergify bot added the ci-passed label Jul 2, 2024
@XuanYang-cn
Copy link
Contributor

/lgtm
/approve

@XuanYang-cn XuanYang-cn added PR | need to cherry-pick to 2.x This PR need to be cherry-picked to 2.x branch and removed approved lgtm labels Jul 2, 2024
@XuanYang-cn
Copy link
Contributor

/lgtm
/approve

@sre-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ashkrisk, XuanYang-cn

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

@sre-ci-robot sre-ci-robot merged commit 9a5e4aa into milvus-io:master Jul 2, 2024
11 checks passed
ashkrisk added a commit to ashkrisk/pymilvus that referenced this pull request Jul 2, 2024
Allows the following operations to be performed through the
`MilvusClient` interface:
- create_database
- drop_database
- list_databases

Resolves milvus-io#2151

Signed-off-by: Ashwin Krishna Kumar <[email protected]>
@XuanYang-cn XuanYang-cn added PR | cherry-picked to 2.x PR already cherry-picked to branch 2.x and removed PR | need to cherry-pick to 2.x This PR need to be cherry-picked to 2.x branch labels Jul 4, 2024
sre-ci-robot pushed a commit that referenced this pull request Jul 4, 2024
Allows the following operations to be performed through the
`MilvusClient` interface:
- create_database
- drop_database
- list_databases

Resolves #2151

Signed-off-by: Ashwin Krishna Kumar <[email protected]>
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.

[Enhancement]: Support database create, list and drop operations for Milvus client
3 participants