Skip to content
This repository has been archived by the owner on Dec 1, 2022. It is now read-only.

fix filter degradation #1240

Merged
merged 5 commits into from
Jul 13, 2021
Merged

fix filter degradation #1240

merged 5 commits into from
Jul 13, 2021

Conversation

czpmango
Copy link
Contributor

To ensure that stableFilter(such as SequentialIter::erase()) is used only if the WITH clause contains ORDER BY.

@czpmango czpmango added the ready-for-testing PR: ready for the CI test label Jul 12, 2021
jievince
jievince previously approved these changes Jul 12, 2021
View reviewed changes
src/planner/match/WithClausePlanner.cpp Show resolved Hide resolved
@czpmango czpmango force-pushed the fix-filter-degradation branch 2 times, most recently from fdc2c7d to 0a8f3e9 Compare July 12, 2021 09:46
@jievince jievince requested a review from Aiee July 13, 2021 02:13
@yixinglu yixinglu merged commit 8a7f48e into vesoft-inc:master Jul 13, 2021
@jamieliu1023 jamieliu1023 mentioned this pull request Jul 17, 2021
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@CPWstatic CPWstatic CPWstatic approved these changes

@Aiee Aiee Aiee approved these changes

@jievince jievince jievince approved these changes

Assignees
No one assigned
Labels
ready-for-testing PR: ready for the CI test
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@czpmango @CPWstatic @Aiee @jievince @yixinglu