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 2.8 crash, and enable load after filter #3808
Conversation
@@ -250,6 +250,53 @@ def testFilter(self): | |||
self.env.assertLess(int(row['count']), 5) | |||
self.env.assertGreater(int(row['count']), 2) | |||
|
|||
def testFilterBeforeLoad(self): |
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.
Previously, this caused a crash
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## master #3808 +/- ##
==========================================
+ Coverage 82.75% 82.80% +0.04%
==========================================
Files 192 192
Lines 32540 32544 +4
==========================================
+ Hits 26930 26948 +18
+ Misses 5610 5596 -14
☔ View full report in Codecov by Sentry. |
* fix 2.8 crash, and enable load after filter * added a test * revert temporary test lines
* fix 2.8 crash, and enable load after filter * added a test * revert temporary test lines
Describe the changes in the pull request
This PR is an alternative fix for #3809 and #3790. It is keeping the behavior of keeping
FILTER
steps in the coordinator but enables other steps that come after aFILTER
step to move to the shards, if possible.This fixes a bug that caused a
LOAD
step after aFILTER
step to stay in the coordinator plan and later caused a crash on 2.8.Which issues this PR fixes
Mark if applicable