-
Notifications
You must be signed in to change notification settings - Fork 3k
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
fix: the close operation of rmq consumer is not sync #38734
Conversation
@chyezh go-sdk check failed, comment |
@chyezh E2e jenkins job failed, comment |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #38734 +/- ##
==========================================
+ Coverage 81.09% 81.13% +0.04%
==========================================
Files 1383 1385 +2
Lines 195636 195831 +195
==========================================
+ Hits 158642 158894 +252
+ Misses 31404 31359 -45
+ Partials 5590 5578 -12
|
d404aef
to
e08434f
Compare
@chyezh E2e jenkins job failed, comment |
Signed-off-by: chyezh <[email protected]>
Signed-off-by: chyezh <[email protected]>
e08434f
to
be3fa87
Compare
@chyezh go-sdk check failed, comment |
@chyezh cpp-unit-test check failed, comment |
rerun cpp-unit-test |
@chyezh E2e jenkins job failed, comment |
/run-cpu-e2e |
@chyezh cpp-unit-test check failed, comment |
1 similar comment
@chyezh cpp-unit-test check failed, comment |
@chyezh go-sdk check failed, comment |
a02e0e0
to
e8858d7
Compare
Signed-off-by: chyezh <[email protected]>
e8858d7
to
4a4cd7a
Compare
rerun cpp-unit-test |
2 similar comments
rerun cpp-unit-test |
rerun cpp-unit-test |
rerun cpp-unit-test |
1 similar comment
rerun cpp-unit-test |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED Approval requirements bypassed by manually added approval. This pull-request has been approved by: chyezh 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 |
issue: #38399