search

apache / arrow

Apache Arrow is a multi-language toolbox for accelerated data interchange and in-memory processing
https://arrow.apache.org/
Apache License 2.0
14.36k stars 3.49k forks source link

[JS] Test JS bundle using parcel #31804

Open asfimport opened 2 years ago

asfimport commented 2 years ago

Currently it can't handle importing like (it can't find map.ts)


import {...} from "apache-arrow"

Also


import {...} from "apache-arrow/src/Arrow.ts"

doesn't work (because it's referring to JS files in the distribution, while the source should refer to .ts files).

 

While the error might be a parcel bug / quirk, how it resolves the paths, we still might to support it's latest versions.

Reporter: Adam Lippai / @alippai

Note: This issue was originally created as ARROW-16429. Please see the migration documentation for further details.

asfimport commented 2 years ago

Adam Lippai / @alippai: If it's caused by this: https://github.com/parcel-bundler/parcel/issues/7823 maybe we should just wait for a fix :/

asfimport commented 2 years ago

Adam Lippai / @alippai: Also if I specify 


scopeHoist: false

 and 


sourceMap: false

 for the target then it works, so this is not major.

domoritz commented 1 year ago

This looks like a bug in Parcel. We can add Parcel to the bundler tests, though.

Btw, https://github.com/vega/falcon uses parcel with arrow and it seems to work. Not sure whether I did anything special but you may want to take a look.