-
Notifications
You must be signed in to change notification settings - Fork 5.9k
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
ranger: BuildColumnRange should merge ranges when column has prefix len #11563
Conversation
Codecov Report
@@ Coverage Diff @@
## master #11563 +/- ##
================================================
+ Coverage 81.2152% 81.3243% +0.1091%
================================================
Files 426 425 -1
Lines 91835 91424 -411
================================================
- Hits 74584 74350 -234
+ Misses 11894 11731 -163
+ Partials 5357 5343 -14 |
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
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.
/run-all-tests |
cherry pick to release-3.0 in PR #11565 |
What problem does this PR solve?
Fix the second case of #11544
What is changed and how it works?
BuildColumnRange
should union the ranges if it has been cut.Check List
Tests
Related changes