Skip to content

feat(load): add bounded Arrow IPC ingestion path - #28145

Open
iamlinjunhong wants to merge 39 commits into
matrixorigin:mainfrom
iamlinjunhong:m-23684
Open

feat(load): add bounded Arrow IPC ingestion path #28145
iamlinjunhong wants to merge 39 commits into
matrixorigin:mainfrom
iamlinjunhong:m-23684

Conversation

@iamlinjunhong

@iamlinjunhong iamlinjunhong commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

What type of PR is this?

  • Feature

Which issue(s) this PR fixes:

Closes #23684

What this PR does / why we need it:

Adds a bounded Apache Arrow IPC File/Stream ingestion path to LOAD DATA. The implementation is LOAD-only and preserves normal statement atomicity, schema validation, range identity checks, and cleanup/lease accounting.

Rollout and compatibility

  • Arrow admission is fail-closed by default: local File/Stream requires cn.frontend.arrow-load.enabled=true; S3/stage and distributed execution additionally require their explicit gates on every executing CN.
  • Remote Arrow pipeline payloads use MORPC v55; v54 peers reject the additive contract.
  • force-materialize=true is the immediate rollback/diagnostic path for borrowed backing.

Design and readiness

The design artifact is an implementation proposal and records that independent approval remains pending; no unapproved Arrow surface is default-reachable. The readiness record also lists the still-required aggregate-admission, provider, mixed-version artifact, deployment, and owner evidence before broader rollout.

Validation

Focused configuration, planner, remote-protocol, external-reader/worker-gate, and real MySQL public-path gate tests pass locally using the repository CGo test wrapper. Required CI is running for the current head.

@qodo-code-review

Copy link
Copy Markdown

Qodo reviews are paused for this user.

Troubleshooting steps vary by plan Learn more →

On a Teams plan?
Reviews resume once this user has a paid seat and their Git account is linked in Qodo.
Link Git account →

Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center?
These require an Enterprise plan - Contact us
Contact us →

@matrix-meow matrix-meow added the size/XXL Denotes a PR that changes 2000+ lines label Sep 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/feature size/XXL Denotes a PR that changes 2000+ lines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature Request]: Load arrow data

5 participants