Skip to content

Fix the issue that the type was not correctly set during secondary index initialization#3279

Merged
yingfeng merged 4 commits intoinfiniflow:0.7.xfrom
euvre:fix-secondary-index-type-bug
Mar 3, 2026
Merged

Fix the issue that the type was not correctly set during secondary index initialization#3279
yingfeng merged 4 commits intoinfiniflow:0.7.xfrom
euvre:fix-secondary-index-type-bug

Conversation

@euvre
Copy link
Contributor

@euvre euvre commented Mar 2, 2026

What problem does this PR solve?

As above.

Type of change

  • Bug Fix (non-breaking change which fixes an issue)

Signed-off-by: noob <[email protected]>
@euvre euvre added the ci PR can be test label Mar 2, 2026
@euvre euvre marked this pull request as ready for review March 2, 2026 11:37
@codecov
Copy link

codecov bot commented Mar 2, 2026

Codecov Report

❌ Patch coverage is 0% with 44 lines in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (0.7.x@a93c528). Learn more about missing BASE report.

Files with missing lines Patch % Lines
...r/file_worker/secondary_index_file_worker_impl.cpp 0.00% 31 Missing ⚠️
src/storage/catalog/meta/chunk_index_meta_impl.cpp 0.00% 13 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff            @@
##             0.7.x    #3279   +/-   ##
========================================
  Coverage         ?   17.49%           
========================================
  Files            ?      729           
  Lines            ?   153378           
  Branches         ?    27750           
========================================
  Hits             ?    26838           
  Misses           ?   120952           
  Partials         ?     5588           
Flag Coverage Δ
debug parallel test 17.49% <0.00%> (?)

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@yingfeng yingfeng merged commit 0c5df9d into infiniflow:0.7.x Mar 3, 2026
30 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci PR can be test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants