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

fix util functions #1277

Merged
merged 1 commit into from
Jul 22, 2021
Merged

fix util functions #1277

merged 1 commit into from
Jul 22, 2021

Conversation

czpmango
Copy link
Contributor

Delete the ObjectPool type parameter in ExpressionUtil functions and fix the inconsistency of the Expression's ObjectPool caused by explicitly specifying the ObjectPool.

@czpmango czpmango added the ready-for-testing PR: ready for the CI test label Jul 21, 2021
@czpmango czpmango requested a review from a team July 21, 2021 09:37
jievince
jievince previously approved these changes Jul 21, 2021
@czpmango czpmango requested a review from a team July 21, 2021 09:56
small delete
@czpmango czpmango requested a review from a team July 22, 2021 02:32
@yixinglu yixinglu merged commit 545aa90 into vesoft-inc:master Jul 22, 2021
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.

None yet

4 participants