parquetjs icon indicating copy to clipboard operation
parquetjs copied to clipboard

Add the ability to fetch remote files (s3 and http[s])

Open ZJONSSON opened this issue 7 years ago • 6 comments

Adapters included for S3 files and files available over http(s). Only the parts if interest are fetched over the wire, eliminating the need to download the complete files. The performance of the adapters when fetching full rows is drastically improved with https://github.com/ironSource/parquetjs/pull/33

No tests so far, but if you are happy with the approach we could add tests using a simple localhost server and an S3 compatible local service, if that makes sense?

ZJONSSON avatar Jan 22 '18 22:01 ZJONSSON

Added the ability to read directly from buffer containing the whole parquet file

ZJONSSON avatar Feb 01 '18 17:02 ZJONSSON

LGTM

asmuth avatar Feb 11 '18 20:02 asmuth

@ZJONSSON let's add tests if you have the time, if not let us know and we'll try to do it

kessler avatar Feb 11 '18 21:02 kessler

I agree. To keep tests as unit tests we would have to add a couple of things to the devDependencies

A test for the buffer reader does not require any additional dependencies

ZJONSSON avatar Feb 12 '18 01:02 ZJONSSON

what is the status of this? abandoned?

Kosta-Github avatar Apr 26 '19 10:04 Kosta-Github

@Kosta-Github it's not abandoned but we're (mainly me) are having problem allocating time for this project, since the initial effort of building it. If you want to contribute, let's discuss :-)

kessler avatar Apr 28 '19 14:04 kessler