-
Notifications
You must be signed in to change notification settings - Fork 960
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
Remove duplicate point calculate because of consensus and so on #12459
base: master
Are you sure you want to change the base?
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #12459 +/- ##
============================================
- Coverage 43.35% 43.13% -0.23%
Complexity 69 69
============================================
Files 3166 3189 +23
Lines 196844 198064 +1220
Branches 23443 23562 +119
============================================
+ Hits 85349 85437 +88
- Misses 111495 112627 +1132 ☔ View full report in Codecov by Sentry. |
Quality Gate failedFailed conditions |
Because we need to know the actual external writing point, I try to add
isGenerateByConsensus
parameters in PlanNode and try to remove duplicate points.