Skip to content

GH-48294: Add RecordBatch#merge#50175

Open
Reranko05 wants to merge 1 commit into
apache:mainfrom
Reranko05:add-record-batch-merge
Open

GH-48294: Add RecordBatch#merge#50175
Reranko05 wants to merge 1 commit into
apache:mainfrom
Reranko05:add-record-batch-merge

Conversation

@Reranko05

@Reranko05 Reranko05 commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

Rationale for this change

This adds Arrow::RecordBatch#merge with behavior analogous to
Arrow::Table#merge.

What changes are included in this PR?

  • Add RecordBatch#merge
  • Support merging with a Hash
  • Support merging with another RecordBatch
  • Support column addition, replacement, and removal (via Hash values of nil)
  • Add tests covering the new behavior

Are these changes tested?

Yes. Added unit tests covering:

  • add column
  • remove column
  • replace column
  • merge with another RecordBatch
  • invalid merge target

Copilot AI review requested due to automatic review settings June 15, 2026 10:59
@Reranko05 Reranko05 requested a review from kou as a code owner June 15, 2026 10:59

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 was unable to review this pull request because the user who requested the review has reached their quota limit.

@github-actions

Copy link
Copy Markdown

⚠️ GitHub issue #48294 has been automatically assigned in GitHub to PR creator.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants