Skip to content
This repository has been archived by the owner on Feb 18, 2024. It is now read-only.

Added support for async parquet write #372

Merged
merged 2 commits into from
Sep 3, 2021
Merged

Added support for async parquet write #372

merged 2 commits into from
Sep 3, 2021

Conversation

GrandChaman
Copy link
Contributor

@GrandChaman GrandChaman commented Sep 3, 2021

This PR adds support for async write to parquet from a Stream of CompressedPages. It is the full async counterpart of arrow2::io::parquet::stream::write_stream, enabling full async writes.

Closes #371

@jorgecarleitao jorgecarleitao added the feature A new feature label Sep 3, 2021
Copy link
Owner

@jorgecarleitao jorgecarleitao left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the interest and for the contribution! Looks great!

@codecov
Copy link

codecov bot commented Sep 3, 2021

Codecov Report

Merging #372 (8af9f85) into main (37353bb) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #372   +/-   ##
=======================================
  Coverage   81.13%   81.13%           
=======================================
  Files         326      326           
  Lines       21150    21150           
=======================================
  Hits        17160    17160           
  Misses       3990     3990           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 37353bb...8af9f85. Read the comment docs.

@jorgecarleitao jorgecarleitao changed the title Added helper function to write asynchronously to a parquet file Added support for async parquet write Sep 3, 2021
@jorgecarleitao jorgecarleitao merged commit f4495b7 into jorgecarleitao:main Sep 3, 2021
@GrandChaman GrandChaman deleted the feature/parquet_async_writes branch September 3, 2021 14:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature A new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Async parquet write
2 participants