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

feat(pisa-proxy, runtime): add rebuild_conn for broken conn in pool. #215

Merged
merged 1 commit into from
Aug 2, 2022

Conversation

xuanyuan300
Copy link
Contributor

Signed-off-by: xuanyuan300 xuanyuan300@gmail.com

What is changed and how it works?

What's Changed:

  1. Add rebuild_conn method for broken conn in pool.

Signed-off-by: xuanyuan300 <xuanyuan300@gmail.com>
@xuanyuan300 xuanyuan300 added app:pisa-proxy Pisa-Proxy related changes in:runtime Runtime related changes labels Aug 2, 2022
@xuanyuan300 xuanyuan300 added this to the v0.2.0 milestone Aug 2, 2022
@xuanyuan300 xuanyuan300 requested a review from wbtlb August 2, 2022 04:11
@codecov
Copy link

codecov bot commented Aug 2, 2022

Codecov Report

❗ No coverage uploaded for pull request base (master@18c8ad3). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #215   +/-   ##
=========================================
  Coverage          ?   30.68%           
=========================================
  Files             ?        3           
  Lines             ?      264           
  Branches          ?        0           
=========================================
  Hits              ?       81           
  Misses            ?      177           
  Partials          ?        6           
Flag Coverage Δ
unittests 30.68% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Help us with your feedback. Take ten seconds to tell us how you rate us.

@wbtlb wbtlb merged commit a48df35 into database-mesh:master Aug 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
app:pisa-proxy Pisa-Proxy related changes in:runtime Runtime related changes size/M
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants