Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

When path query limit, response time is too slow #2506

Open
Topcoder-HD opened this issue Aug 12, 2021 · 2 comments
Open

When path query limit, response time is too slow #2506

Topcoder-HD opened this issue Aug 12, 2021 · 2 comments
Labels
community Source: who proposed the issue type/enhancement Type: make the code neat or more efficient

Comments

@Topcoder-HD
Copy link

find path 和match路径查询在进行limit的时候速度很慢，我发现目前的limit的策略是把全部的结果查询出来后再进行limit，这样的话其实仍然会执行全部查询，所以查询响应时间很慢，所以我建议能否在查询之前进行limit，类似neo4j的limit查询，这样速度会快些
@wey-gu
Copy link
Contributor

wey-gu commented Aug 12, 2021

多谢您 @Topcoder-HD ，limit 在这些情况的下推（storage pushdown）优化我们正在进行中哈

@jamieliu1023 jamieliu1023 mentioned this issue Aug 14, 2021
Closed
@CPWstatic CPWstatic added type/enhancement Type: make the code neat or more efficient type/feature req Type: feature request labels Aug 27, 2021
@Sophie-Xie Sophie-Xie removed the type/feature req Type: feature request label Aug 30, 2021
@Sophie-Xie Sophie-Xie changed the title 路径查询limit响应时间太慢问题 When path query limit, response time is too slow Sep 6, 2021
@Sophie-Xie Sophie-Xie added the community Source: who proposed the issue label Sep 13, 2021
@Sophie-Xie
Copy link
Contributor

Some of the most import computation need to push down:

limit
sample
existed bug in filter
Three above should be complete in 2.6.0. #2602

GroupBy/OrderBy/DeDup/Aggregation's detail need to discuss later if necessary.

yixinglu pushed a commit to yixinglu/nebula that referenced this issue Sep 14, 2023
@nebula-bots @critical27 @Sophie-Xie
Backport some enhancement from poc branch (vesoft-inc#2506)
9475423 
* only retrieve schema once when insert

* add limit by gflag in GetDstBySrc

* improve replace host by http

---------

Co-authored-by: Doodle <13706157+critical27@users.noreply.github.com>
Co-authored-by: Sophie <84560950+Sophie-Xie@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
community Source: who proposed the issue type/enhancement Type: make the code neat or more efficient
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@wey-gu @CPWstatic @Topcoder-HD @Sophie-Xie