Skip to content

feat: impl Refresh Ngram Index - #18128

Merged
bohutang merged 5 commits into
databendlabs:mainfrom
KKould:feat/refresh_ngram_index
Jun 16, 2025
Merged

feat: impl Refresh Ngram Index#18128
bohutang merged 5 commits into
databendlabs:mainfrom
KKould:feat/refresh_ngram_index

Conversation

@KKould

@KKould KKould commented Jun 10, 2025

Copy link
Copy Markdown
Member

I hereby agree to the terms of the CLA available at: https://docs.databend.com/dev/policies/cla/

Summary

  • REFRESH NGRAM INDEX <index_name> ON <table_name>

Used to support reloading index data after creating a new Ngram Index

Tests

  • Unit Test
  • Logic Test
  • Benchmark Test
  • No Test - Explain why

Type of change

  • Bug Fix (non-breaking change which fixes an issue)
  • New Feature (non-breaking change which adds functionality)
  • Breaking Change (fix or feature that could cause existing functionality not to work as expected)
  • Documentation Update
  • Refactoring
  • Performance Improvement
  • Other (please describe):

This change is Reviewable

@github-actions github-actions Bot added the pr-feature this PR introduces a new feature to the codebase label Jun 10, 2025
@KKould
KKould force-pushed the feat/refresh_ngram_index branch 3 times, most recently from 748ad34 to 2bf20f9 Compare June 10, 2025 14:16
@KKould
KKould marked this pull request as ready for review June 10, 2025 15:49
@b41sh
b41sh self-requested a review June 11, 2025 03:49
Comment thread src/query/ee/src/storages/fuse/operations/ngram_index.rs Outdated
Comment thread src/query/ee/src/storages/fuse/operations/ngram_index.rs Outdated
Comment thread src/query/ee/src/storages/fuse/operations/ngram_index.rs Outdated
Comment thread src/query/ee/src/storages/fuse/operations/ngram_index.rs Outdated
@KKould
KKould force-pushed the feat/refresh_ngram_index branch from 6479227 to 19f7454 Compare June 11, 2025 09:14
Comment thread src/query/ee/src/storages/fuse/operations/ngram_index.rs Outdated
Comment thread src/query/ee/src/storages/fuse/operations/ngram_index.rs Outdated
@KKould
KKould force-pushed the feat/refresh_ngram_index branch 8 times, most recently from c049704 to 585ed99 Compare June 13, 2025 04:24
@KKould
KKould force-pushed the feat/refresh_ngram_index branch 2 times, most recently from c38f1b2 to b153670 Compare June 14, 2025 01:23
@KKould
KKould force-pushed the feat/refresh_ngram_index branch from b153670 to d6a93c1 Compare June 14, 2025 03:06
@bohutang
bohutang merged commit 71c25f5 into databendlabs:main Jun 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-feature this PR introduces a new feature to the codebase

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants