Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use Flechette for Apache Arrow processing #362

Merged
merged 16 commits into from
Sep 17, 2024
Merged

Use Flechette for Apache Arrow processing #362

merged 16 commits into from
Sep 17, 2024

Conversation

jheer
Copy link
Member

@jheer jheer commented Aug 13, 2024

This PR changes the default handler of Apache Arrow data from the apache-arrow JS reference implementation to the smaller, faster, and more flexible @uwdata/flechette. The change improves performance a bit and simplifies Arquero internals, as it obviates the need for internal custom Arrow data conversion steps and a variety of earlier performance and correctness workarounds. Arrow IPC binary decoding (in fromArrow) and building/encoding (in toArrow and toArrowIPC) now use Flechette.

If passing an already instantiated Arrow table to fromArrow, Arquero supports both apache-arrow and @uwdata/flechette tables, including optimizations for groupby and filter operations over dictionary types. However, special extraction of decimals, dates, and nested types (lists, structs) is no longer provided by Arquero; instead we default to whatever the Arrow implementation provides. For Flechette, this includes customizable extraction options and efficient serialization to native JS array and object types. For Arrow-JS, decimals are not well-supported and lists are returned as Arrow Vector types and structs as slow Proxy row objects.

This PR makes breaking changes. If loading data directly from IPC bytes, Arquero results should be consistent. Moreover, as Flechette is small, it is bundled directly with Arquero, so no additional imports are needed. However, when passing-in already-parsed apache-arrow tables, results will differ for some data types.

@jheer jheer added the enhancement New feature or request label Aug 13, 2024
@jheer jheer marked this pull request as ready for review August 26, 2024 14:59
@jheer jheer changed the title Use Flechette as the default Arrow decoder Use Flechette for Apache Arrow data processing Sep 13, 2024
@jheer jheer changed the title Use Flechette for Apache Arrow data processing Use Flechette for Apache Arrow processing Sep 13, 2024
@jheer jheer merged commit c60f694 into main Sep 17, 2024
2 checks passed
@jheer jheer deleted the jh/flechette branch September 17, 2024 02:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant