Skip to content
This repository was archived by the owner on Apr 27, 2026. It is now read-only.

feat: Move from nanoarrow to arrow c++ as data container backend#316

Merged
xmnlab merged 2 commits intoarxlang:mainfrom
xmnlab:move-from-nanoarrow-to-arrowcpp
Apr 25, 2026
Merged

feat: Move from nanoarrow to arrow c++ as data container backend#316
xmnlab merged 2 commits intoarxlang:mainfrom
xmnlab:move-from-nanoarrow-to-arrowcpp

Conversation

@xmnlab
Copy link
Copy Markdown
Contributor

@xmnlab xmnlab commented Apr 25, 2026

Pull Request description

Note: Arrow C++ source/header metadata comes from arx-arrowcpp-sources; local linking uses PyArrow’s
installed Arrow C++ shared library to avoid compiling the full Arrow C++ source tree during IRx
runtime builds.

How to test these changes

  • ...

Pull Request checklists

This PR is a:

  • bug-fix
  • new feature
  • maintenance

About this PR:

  • it includes tests.
  • the tests are executed on CI.
  • the tests generate log file(s) (path).
  • pre-commit hooks were executed locally.
  • this PR requires a project documentation update.

Author's checklist:

  • I have reviewed the changes and it contains no misspelling.
  • The code is well commented, especially in the parts that contain more
    complexity.
  • New and old tests passed locally.

Additional information

Reviewer's checklist

Copy and paste this template for your review's note:

## Reviewer's Checklist

- [ ] I managed to reproduce the problem locally from the `main` branch
- [ ] I managed to test the new changes locally
- [ ] I confirm that the issues mentioned were fixed/resolved .

@xmnlab xmnlab merged commit ac1622d into arxlang:main Apr 25, 2026
13 checks passed
@xmnlab xmnlab deleted the move-from-nanoarrow-to-arrowcpp branch April 25, 2026 05:23
@github-actions
Copy link
Copy Markdown

🎉 This PR is included in version 1.18.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant