Skip to content

refactor(lumina): extract shared index components - #383

Open
zjw1111 wants to merge 1 commit into
apache:mainfrom
zjw1111:codex/lumina-index-refactor
Open

zjw1111 wants to merge 1 commit into
apache:mainfrom
zjw1111:codex/lumina-index-refactor

Conversation

@zjw1111

@zjw1111 zjw1111 commented Sep 23, 2026

Copy link
Copy Markdown
Collaborator

Purpose

Linked issue: #361

Move the Lumina plugin from src/paimon/global_index/lumina to src/paimon/indexer/lumina and separate the Global Index adapter from the Lumina-specific reusable components.

This change extracts the shared dataset, index accumulator, option parsing, tag conversion, and search lifecycle code. LuminaIndexSearcher owns the opened native searcher and its extensions, performs vector-search dispatch, and closes the native searcher through RAII. The existing Lumina Global Index behavior remains unchanged.

This PR does not add a Lumina File Index implementation or change the Paimon File Index and Global Index framework APIs.

Tests

  • cmake --build build --target paimon-lumina-index-test -j 16
  • ./build/debug/paimon-lumina-index-test --gtest_brief=1 (22 tests passed)
  • pre-commit run --files <changed files>
  • git diff --check

API and Format

No public API, storage format, or protocol changes.

Documentation

No user-facing feature or documentation change.

Generative AI tooling

Generated-by: OpenAI Codex (GPT-5)

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant