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

Update README.md with a full stream process #175

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

pbrink231
Copy link

Added a streaming method to process csv line items without loading into memory

This PR contains:

  • bugfix
  • feature
  • refactor
  • tests
  • documentation
  • metadata

Breaking Changes?

  • yes
  • no

If yes, please describe the breakage.

Please Describe Your Changes

This will show a way to process a csv line by line without loading it into memory. Very useful for large files and allows async/await on each line. Used to contact a database or third party api with the information in the line and pause the process while it happens.

Added a streaming method to process csv line items without loading into memory
@codecov-io
Copy link

Codecov Report

Merging #175 (4b5492a) into master (dd5123b) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #175   +/-   ##
=======================================
  Coverage   97.20%   97.20%           
=======================================
  Files           1        1           
  Lines         143      143           
=======================================
  Hits          139      139           
  Misses          4        4           

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 dd5123b...4b5492a. Read the comment docs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants