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

fix ci crash #1298

Merged
merged 1 commit into from Jul 30, 2021
Merged

fix ci crash #1298

merged 1 commit into from Jul 30, 2021

Conversation

czpmango
Copy link
Contributor

The CI crash is caused by the empty inputVar on the plan, which is triggered by CollapseProjectRule.

This pr fixes inputVar of the first plan node.

@czpmango czpmango added do not review PR: not ready for the code review yet ready-for-testing PR: ready for the CI test labels Jul 30, 2021
@czpmango czpmango removed the do not review PR: not ready for the code review yet label Jul 30, 2021
@czpmango czpmango requested review from a team, CPWstatic and Shylock-Hg July 30, 2021 10:11
@yixinglu yixinglu merged commit 2669b9f into vesoft-inc:master Jul 30, 2021
@czpmango czpmango mentioned this pull request Jul 30, 2021
@czpmango czpmango deleted the fix-crash branch August 2, 2021 02:02
@Shylock-Hg Shylock-Hg linked an issue Aug 2, 2021 that may be closed by this pull request
CPWstatic pushed a commit to CPWstatic/nebula-graph that referenced this pull request Aug 2, 2021
CPWstatic added a commit that referenced this pull request Aug 3, 2021
* Fix the go return bad_type when the vertex without the tag (#1273)

Co-authored-by: Yee <2520865+yixinglu@users.noreply.github.com>

* fix util functions (#1277)

small delete

* Check whether index is valid at runtime (#1291)

* Check whether index is valid at runtime

* Fix failed ut

* Fix steps

* drop the used space

* Format

* Replace the lastUser by user count.… (#1243)

* Replace the lastUser by user count.The last user maybe could run simultaneously, so can't determine which is real last user in timeline.

* Fix the typo.

Co-authored-by: Yee <2520865+yixinglu@users.noreply.github.com>

* fix first plan node input var (#1298)

Co-authored-by: laura-ding <48548375+laura-ding@users.noreply.github.com>
Co-authored-by: Yee <2520865+yixinglu@users.noreply.github.com>
Co-authored-by: kyle.cao <kyle.cao@vesoft.com>
Co-authored-by: Shylock Hg <33566796+Shylock-Hg@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.

Crash when collapse project optimize
3 participants