-
Notifications
You must be signed in to change notification settings - Fork 3.2k
enhance: add delete rows into delete msg header and more metric #41952
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: add delete rows into delete msg header and more metric #41952
Conversation
98cbb03
to
be74170
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: chyezh, czs007 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 |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #41952 +/- ##
===========================================
+ Coverage 73.04% 80.46% +7.42%
===========================================
Files 335 1537 +1202
Lines 30702 216752 +186050
===========================================
+ Hits 22425 174407 +151982
- Misses 8277 36055 +27778
- Partials 0 6290 +6290
🚀 New features to boost your workflow:
|
be74170
to
0ee62f6
Compare
New changes are detected. LGTM label has been removed. |
@chyezh E2e jenkins job failed, comment |
0ee62f6
to
7d32e6c
Compare
@chyezh E2e jenkins job failed, comment |
7d32e6c
to
39d6ef9
Compare
- add delete rows into delete messsage header - add more insert/delete metrics - fix non-broadcast message has broadcast header Signed-off-by: chyezh <[email protected]>
39d6ef9
to
afbd17a
Compare
@chyezh E2e jenkins job failed, comment |
/run-cpu-e2e |
rerun ut |
issue: #41544