Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

CPWstatic
Copy link
Contributor

No description provided.

@CPWstatic CPWstatic added the ready-for-testing PR: ready for the CI test label Jan 22, 2020
@knightXun
Copy link
Contributor

related to issue: #1699

@jude-zhu
Copy link
Contributor

@knightXun we need more test cases 😁😁

Copy link
Contributor

@monadbobo monadbobo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@dutor dutor merged commit 8a9daec into vesoft-inc:master Feb 2, 2020
jude-zhu pushed a commit to jude-zhu/nebula that referenced this pull request Feb 3, 2020
Co-authored-by: dutor <[email protected]>
(cherry picked from commit 8a9daec)
jude-zhu pushed a commit to jude-zhu/nebula that referenced this pull request Feb 3, 2020
Co-authored-by: dutor <[email protected]>
(cherry picked from commit 8a9daec)
(cherry picked from commit 65364c2)
@jude-zhu jude-zhu mentioned this pull request Feb 3, 2020
dutor pushed a commit that referenced this pull request Feb 3, 2020
Co-authored-by: dutor <[email protected]>
(cherry picked from commit 8a9daec)
(cherry picked from commit 65364c2)

Co-authored-by: CPWstatic <[email protected]>
tong-hao pushed a commit to tong-hao/nebula that referenced this pull request Jun 1, 2021
yixinglu pushed a commit to yixinglu/nebula that referenced this pull request Jan 31, 2023
…er push-down rules. (vesoft-inc#1700)

- Added some tck tests.

Co-authored-by: Cheng Xuntao <[email protected]>
yixinglu pushed a commit to yixinglu/nebula that referenced this pull request Jan 31, 2023
yixinglu pushed a commit to yixinglu/nebula that referenced this pull request Jan 31, 2023
…esoft-inc#1709)

* - Fix the setting of colNames and exchanging of PlanNodes in two filter push-down rules. (vesoft-inc#1700)

- Added some tck tests.

Co-authored-by: Cheng Xuntao <[email protected]>

* Fix the setting of colNames in 3 optimizers' rules.

Co-authored-by: nebula-bots <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-for-testing PR: ready for the CI test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants