Skip to content

Conversation

@lxy-9602
Copy link
Collaborator

@lxy-9602 lxy-9602 commented Jan 26, 2026

Purpose

Integrate Lucene++ to enable full-text search capabilities in Paimon.
Support basic query types: MATCH_ALL, MATCH_ANY, PHRASE, PREFIX, and WILDCARD.

Linked issue: #69

Key changes:

  • Add FullTextSearch struct with search type enum and parameters
  • Use Lucene::Directory wrapper for safe access to Paimon file system
  • Support index write and search operations

Thank you to @wuxueyang96 from StarRocks for the helpful inspiration on the full-text search design.

TODO: Implement jieba analyzer

Tests

LuceneInterfaceTest
LuceneGlobalIndexTest

API and Format

FullTextSearch

Documentation

@lucasfang lucasfang requested a review from Copilot January 27, 2026 02:44
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@lxy-9602 lxy-9602 changed the title feat: support full text search with Lucene++ feat: support full-text search with Lucene++ Jan 29, 2026
@lucasfang lucasfang requested a review from Copilot January 29, 2026 00:57
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 25 out of 25 changed files in this pull request and generated 7 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Collaborator

@lucasfang lucasfang left a comment

Choose a reason for hiding this comment

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

+1

@lucasfang lucasfang merged commit a6349c3 into alibaba:main Jan 29, 2026
8 checks passed
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.

2 participants