Skip to content

Comments

fix: Fix a bug of reset connection#3085

Merged
sre-ci-robot merged 1 commit intomilvus-io:2.6from
yhmo:26
Nov 20, 2025
Merged

fix: Fix a bug of reset connection#3085
sre-ci-robot merged 1 commit intomilvus-io:2.6from
yhmo:26

Conversation

@yhmo
Copy link
Contributor

@yhmo yhmo commented Nov 20, 2025

issue: #3066

@yhmo yhmo changed the title Fix a bug of reset connection fix: Fix a bug of reset connection Nov 20, 2025
@mergify mergify bot added the dco-passed label Nov 20, 2025
@codecov
Copy link

codecov bot commented Nov 20, 2025

Codecov Report

❌ Patch coverage is 22.22222% with 42 lines in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (2.6@3da1040). Learn more about missing BASE report.

Files with missing lines Patch % Lines
pymilvus/client/grpc_handler.py 21.15% 41 Missing ⚠️
pymilvus/orm/connections.py 50.00% 1 Missing ⚠️
Additional details and impacted files
@@          Coverage Diff           @@
##             2.6    #3085   +/-   ##
======================================
  Coverage       ?   45.07%           
======================================
  Files          ?       65           
  Lines          ?    13772           
  Branches       ?        0           
======================================
  Hits           ?     6208           
  Misses         ?     7564           
  Partials       ?        0           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@yhmo yhmo force-pushed the 26 branch 2 times, most recently from d5c47e4 to 933991d Compare November 20, 2025 07:38
@mergify mergify bot added the ci-passed label Nov 20, 2025
Copy link
Contributor

@XuanYang-cn XuanYang-cn left a comment

Choose a reason for hiding this comment

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

/lgtm

@sre-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: XuanYang-cn, 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

@sre-ci-robot sre-ci-robot merged commit 28f713c into milvus-io:2.6 Nov 20, 2025
13 checks passed
@yhmo yhmo deleted the 26 branch December 2, 2025 06: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.

3 participants