GeorgeLeePatterson opened a new pull request, #324: URL: https://github.com/apache/arrow-js/pull/324
## What's Changed This PR adds write support for ListView and LargeListView types through builder classes, completing the view types implementation. ## Implementation Details ### Builders - ListViewBuilder and LargeListViewBuilder in `src/builder/listview.ts` - Uses Int32Array (ListView) and BigInt64Array (LargeListView) for offsets/sizes - Custom flush() to pass both valueOffsets and sizes to makeData - BigInt(0) syntax for ES5 compatibility (not 0n literals) ### Integration - Added to builderctor visitor for factory support - Exported from Arrow.ts and Arrow.dom.ts for public API ## What Works - [X] Building ListView/LargeListView columns from JavaScript arrays - [X] Separate offset and size tracking - [X] Proper null handling - [X] Round-trip: build → flush → read ## Testing - [X] Comprehensive unit tests in test/unit/builders/listview-tests.ts - [X] Tests verify offset/size semantics, null handling - [X] All existing tests pass - [X] CI validated on fork Builds on #320 and the ListView read support PR -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
