Skip to content

fix several bugs for performance #730

Merged
ShawnShawnYou merged 5 commits into
mainfrom
fix-several-bugs-for-performance
May 24, 2025
Merged

fix several bugs for performance #730
ShawnShawnYou merged 5 commits into
mainfrom
fix-several-bugs-for-performance

Conversation

@ShawnShawnYou

@ShawnShawnYou ShawnShawnYou commented May 19, 2025

Copy link
Copy Markdown
Collaborator

close #731

@ShawnShawnYou ShawnShawnYou self-assigned this May 19, 2025
@ShawnShawnYou
ShawnShawnYou force-pushed the fix-several-bugs-for-performance branch from 767b37b to 767ad02 Compare May 19, 2025 12:50

@inabao inabao left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

Signed-off-by: zhongxiaoyao.zxy <zhongxiaoyao.zxy@antgroup.com>
Signed-off-by: zhongxiaoyao.zxy <zhongxiaoyao.zxy@antgroup.com>
Signed-off-by: zhongxiaoyao.zxy <zhongxiaoyao.zxy@antgroup.com>
@ShawnShawnYou
ShawnShawnYou force-pushed the fix-several-bugs-for-performance branch from 6df3e7c to 0568ba0 Compare May 21, 2025 02:57
Signed-off-by: zhongxiaoyao.zxy <zhongxiaoyao.zxy@antgroup.com>
Signed-off-by: zhongxiaoyao.zxy <zhongxiaoyao.zxy@antgroup.com>
@codecov

codecov Bot commented May 23, 2025

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

@@            Coverage Diff             @@
##             main     #730      +/-   ##
==========================================
+ Coverage   90.95%   91.05%   +0.09%     
==========================================
  Files         190      190              
  Lines       11787    11867      +80     
==========================================
+ Hits        10721    10805      +84     
+ Misses       1066     1062       -4     
Flag Coverage Δ
cpp 91.05% <100.00%> (+0.09%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Components Coverage Δ
common 92.81% <ø> (+0.16%) ⬆️
datacell 92.38% <100.00%> (-0.01%) ⬇️
index 88.72% <100.00%> (+0.11%) ⬆️
simd 100.00% <ø> (ø)

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 67878bf...8b5c5fd. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@LHT129 LHT129 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@ShawnShawnYou
ShawnShawnYou merged commit 5146774 into main May 24, 2025
@ShawnShawnYou
ShawnShawnYou deleted the fix-several-bugs-for-performance branch May 24, 2025 16:34
dasurax pushed a commit to dasurax/vsag that referenced this pull request Jun 9, 2025
* fix several bugs for performance in sq and optimizer

Signed-off-by: zhongxiaoyao.zxy <zhongxiaoyao.zxy@antgroup.com>
Signed-off-by: suguan.dx <suguan.dx@antgroup.con>
LHT129 pushed a commit to LHT129/vsag that referenced this pull request Apr 16, 2026
* fix several bugs for performance in sq and optimizer

Signed-off-by: zhongxiaoyao.zxy <zhongxiaoyao.zxy@antgroup.com>
LHT129 pushed a commit that referenced this pull request May 11, 2026
* fix several bugs for performance in sq and optimizer

Signed-off-by: zhongxiaoyao.zxy <zhongxiaoyao.zxy@antgroup.com>
Signed-off-by: LHT129 <tianlan.lht@antgroup.com>
Sia-Sheerland pushed a commit to Sia-Sheerland/vsag that referenced this pull request Jun 26, 2026
* fix several bugs for performance in sq and optimizer

Signed-off-by: zhongxiaoyao.zxy <zhongxiaoyao.zxy@antgroup.com>
Signed-off-by: Sia Sheerland <x1075956441x@163.com>

Signed-off-by: Sia Sheerland <x1075956441x@163.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

there's performance issue when setting specific hgraph config

3 participants