Skip to content

fix(predicate): export Like symbols - #391

Merged
lxy-9602 merged 4 commits into
apache:mainfrom
qusijun:fix/export-like-vtable
Sep 27, 2026
Merged

lxy-9602 merged 4 commits into
apache:mainfrom
qusijun:fix/export-like-vtable

Conversation

@qusijun

@qusijun qusijun commented Sep 25, 2026 •

Copy link
Copy Markdown
Contributor

Purpose

Export a narrow MatchString function from paimon_shared and use it from paimon_global_index_shared. The bitmap reader can evaluate LIKE predicates without exporting or referencing the internal polymorphic Like class hierarchy across shared libraries.

Tests

  • Failure reproduced on macOS 14.4.1 arm64 with AppleClang 15.0.0.15000309, CMake 4.4.0, and PAIMON_BUILD_SHARED=ON.
  • After this fix, cmake --build build --target paimon_global_index_shared -j 8 passed on the same environment.
  • Linux CI exposed unresolved base-class vtable/typeinfo references when the first fix exported Like; this follow-up keeps that hierarchy internal.
  • No unit or integration tests were run; this change addresses shared-library symbol visibility.

API and Format

No public header under include/ or storage format/protocol changes. The internal predicate helper is exported for cross-library use.

Documentation

No documentation update needed.

Generative AI tooling

Generated-by: OpenAI Codex (GPT-6)

Copilot AI 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.

Copilot review overview

🟢 Approval recommended

No unresolved review issues were identified.

Review effort: Lite
Findings: None

What changed in this PR

Exports paimon::Like so shared-library builds can resolve its vtable.

Changes:

  • Includes visibility macros.
  • Marks Like with PAIMON_EXPORT.
File Description
src/​paimon/​common/​predicate/​like.h Exports Like across shared-library boundaries.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread src/paimon/common/predicate/like.h Outdated

@lxy-9602 lxy-9602 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

+1

@lxy-9602
lxy-9602 merged commit 7e18577 into apache:main Sep 27, 2026
16 checks passed
@qusijun
qusijun deleted the fix/export-like-vtable branch September 28, 2026 00:00
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.

3 participants