Skip to content

feat: introduce list partitions in cpp binding#260

Merged
luoyuxia merged 1 commit intoapache:mainfrom
luoyuxia:support-list-partitino-info-cpp-binding
Feb 6, 2026
Merged

feat: introduce list partitions in cpp binding#260
luoyuxia merged 1 commit intoapache:mainfrom
luoyuxia:support-list-partitino-info-cpp-binding

Conversation

@luoyuxia
Copy link
Contributor

@luoyuxia luoyuxia commented Feb 6, 2026

Purpose

Linked issue: close #259

Brief change log

Tests

API and Format

Documentation

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

This PR introduces support for listing partition information in the C++ bindings, addressing issue #259. It exposes the existing list_partition_infos functionality from the core Rust library through the C++ FFI layer, allowing C++ clients to query partition metadata (partition ID and name) for a given table.

Changes:

  • Added FFI structures (FfiPartitionInfo, FfiListPartitionInfosResult) and method signature for listing partition information
  • Implemented the Rust-side FFI function that calls the core admin client's list_partition_infos method
  • Added C++ wrapper function ListPartitionInfos in the Admin class that handles FFI conversions and error handling

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
bindings/cpp/src/lib.rs Defines FFI structures for partition info and implements the Rust-side FFI bridge function that calls the core admin API
bindings/cpp/src/admin.cpp Implements the C++ wrapper ListPartitionInfos method with proper error handling and type conversions
bindings/cpp/include/fluss.hpp Declares the PartitionInfo struct and ListPartitionInfos public API method in the Admin class

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

@luoyuxia luoyuxia merged commit 8b583a8 into apache:main Feb 6, 2026
19 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.

support list partition info in cpp binding

1 participant