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

fix match agg check #919

Merged
merged 2 commits into from
Apr 7, 2021
Merged

fix match agg check #919

merged 2 commits into from
Apr 7, 2021

Conversation

czpmango
Copy link
Contributor

@czpmango czpmango commented Apr 6, 2021

Add error check for aggregate expression in match where clause.

@czpmango czpmango added wip Solution: work in progress do not review PR: not ready for the code review yet ready-for-testing PR: ready for the CI test labels Apr 6, 2021
jievince
jievince previously approved these changes Apr 6, 2021
jievince
jievince previously approved these changes Apr 6, 2021
yixinglu
yixinglu previously approved these changes Apr 6, 2021
@czpmango czpmango added ready-for-review and removed wip Solution: work in progress do not review PR: not ready for the code review yet labels Apr 6, 2021
@czpmango czpmango requested review from yixinglu, jievince and a team April 6, 2021 09:32
@yixinglu yixinglu merged commit e6cf4b2 into vesoft-inc:master Apr 7, 2021
jude-zhu pushed a commit to jude-zhu/nebula-graph that referenced this pull request Apr 7, 2021
add ut

add tck

Co-authored-by: jie.wang <38901892+jievince@users.noreply.github.com>
(cherry picked from commit e6cf4b2)
jude-zhu pushed a commit to jude-zhu/nebula-graph that referenced this pull request Apr 7, 2021
add ut

add tck

Co-authored-by: jie.wang <38901892+jievince@users.noreply.github.com>
(cherry picked from commit e6cf4b2)
yixinglu added a commit that referenced this pull request Apr 8, 2021
* fix agg executor when input dataset is empty (#901)

add tck

fix tck

format

fix sum(null) test cases

format

(cherry picked from commit e8ad874)

* Fix match where agg check (#919)

add ut

add tck

Co-authored-by: jie.wang <38901892+jievince@users.noreply.github.com>
(cherry picked from commit e6cf4b2)

* Fix cluster getNeightborIter (#922)

* fix cluster getNeightborIter

* add test case

Co-authored-by: cpw <13495049+CPWstatic@users.noreply.github.com>
(cherry picked from commit 8063659)

* Fix seek by negative vid (#932)

* Fix seek by negtive vid

Fix typo

* Fix tck

(cherry picked from commit e70c326)

* Retry for queries related to schema modifications in CI (#849)

* Retry and wait job finish

* cleanup lookup test wait time

* Fix test failures

* Drop unused spaces

* Cleanup wait tag edge indexes ready function

* Fix cases

* Drop the created spaces

* Result should not contains

* Fix contains steps

* Rename include to contains

* Use more threads to run tests

* Fix typo

* Address ding's comments

* Rename to cmpType

* Try to replace wait

(cherry picked from commit 3c66147)

Co-authored-by: kyle.cao <kyle.cao@vesoft.com>
Co-authored-by: jimingquan <mingquan.ji@vesoft.com>
Co-authored-by: Yichen Wang <18348405+Aiee@users.noreply.github.com>
Co-authored-by: Yee <2520865+yixinglu@users.noreply.github.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
ready-for-testing PR: ready for the CI test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants