Skip to content

Add Arrow CSR query APIs#18

Merged
adsharma merged 1 commit into
mainfrom
feature/arrow-csr-nodejs
May 24, 2026
Merged

Add Arrow CSR query APIs#18
adsharma merged 1 commit into
mainfrom
feature/arrow-csr-nodejs

Conversation

@adsharma
Copy link
Copy Markdown
Contributor

@adsharma adsharma commented May 23, 2026

Summary

  • Add Node.js queryArrow/queryArrowSync APIs backed by Ladybug queryAsArrow.
  • Add ArrowQueryResult.csr() for zero-copy CSR arrays matching the Python binding shape.
  • Add apache-arrow as a package dependency for Arrow-facing Node workflows.
  • Add CSR test coverage for sync, async, rel-id, no-rel-id, Apache Arrow vector wrapping, and non-CSR rejection cases.

Validation

  • CC=/opt/homebrew/opt/llvm/bin/clang CXX=/opt/homebrew/opt/llvm/bin/clang++ npm run build
  • npm test -- test/test.js --grep "Arrow query CSR"
  • npm test
  • node -c src_js/query_result.js
  • node -c src_js/connection.js
  • git diff --check

@adsharma adsharma force-pushed the feature/arrow-csr-nodejs branch 2 times, most recently from 69a5fdc to 9251676 Compare May 23, 2026 23:20
@adsharma adsharma force-pushed the feature/arrow-csr-nodejs branch from 9251676 to 766df7b Compare May 24, 2026 00:09
@adsharma adsharma marked this pull request as ready for review May 24, 2026 05:00
@adsharma adsharma merged commit ef0710c into main May 24, 2026
0 of 8 checks passed
@adsharma adsharma deleted the feature/arrow-csr-nodejs branch May 24, 2026 05:01
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.

1 participant