feat: add search predicates and PredicateBuilder and tests by lxy-9602 · Pull Request #36 · apache/paimon-cpp · GitHub
Skip to content

feat: add search predicates and PredicateBuilder and tests - #36

Merged
leaves12138 merged 2 commits into
apache:mainfrom
lxy-9602:add-predicate-test
Jun 2, 2026
Merged

feat: add search predicates and PredicateBuilder and tests#36
leaves12138 merged 2 commits into
apache:mainfrom
lxy-9602:add-predicate-test

Conversation

@lxy-9602

@lxy-9602 lxy-9602 commented Jun 1, 2026

Copy link
Copy Markdown
Member

Purpose

No Linked issue.

Complete the predicate framework with builder, utilities, validation, and search predicates:

  • PredicateBuilder — fluent API to construct leaf/compound predicates against a RowType (predicate_builder.h/cpp)
  • PredicateUtils — helpers for predicate transformation, splitting by And, and field extraction (predicate_utils.h/cpp)
  • PredicateValidator — validates predicate field references and type compatibility (predicate_validator.h)
  • FullTextSearch — full-text search predicate descriptor (full_text_search.h)
  • VectorSearch — vector similarity search predicate descriptor (vector_search.h)

Tests

  • predicate_test.cpp — end-to-end predicate construction and evaluation across all operators
  • predicate_utils_test.cpp — predicate transformation and splitting
  • predicate_validator_test.cpp — field reference and type validation

API and Format

New public headers under include/paimon/predicate/ (predicate_builder.h, predicate_utils.h, full_text_search.h, vector_search.h).

Documentation

Generative AI tooling

Migrate-by: Aone Copilot (Claude)

@lxy-9602

lxy-9602 commented Jun 1, 2026

Copy link
Copy Markdown
Member Author

@leaves12138 leaves12138 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Reviewed the latest head. The predicate builder/utils/validator, search predicate descriptors, and migrated predicate tests look good to me. I did not find blockers in this pass.

@leaves12138
leaves12138 merged commit 445feeb into apache:main Jun 2, 2026
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