Skip to content

fix: read entire query response into bytes array#254

Merged
bednar merged 2 commits intomasterfrom
fix/streaming-query
Aug 6, 2021
Merged

fix: read entire query response into bytes array#254
bednar merged 2 commits intomasterfrom
fix/streaming-query

Conversation

@bednar
Copy link
Copy Markdown
Contributor

@bednar bednar commented Aug 5, 2021

Closes #253

Proposed Changes

Avoid to read entire query response into byte arrays - https://square.github.io/retrofit/2.x/retrofit/retrofit2/http/Streaming.html

Checklist

  • CHANGELOG.md updated
  • Rebased/mergeable
  • mvn test completes successfully
  • Commit messages are in semantic format
  • Sign CLA (if not already signed)

@bednar bednar changed the title fix: read entire query response into byte arrays fix: read entire query response into bytes array Aug 5, 2021
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Aug 5, 2021

Codecov Report

Merging #254 (95789ef) into master (a95431c) will not change coverage.
The diff coverage is n/a.

❗ Current head 95789ef differs from pull request most recent head 38c4dff. Consider uploading reports for the commit 38c4dff to get more accurate results
Impacted file tree graph

@@            Coverage Diff            @@
##             master     #254   +/-   ##
=========================================
  Coverage     89.26%   89.26%           
  Complexity      449      449           
=========================================
  Files           145      145           
  Lines          5767     5767           
  Branches        271      271           
=========================================
  Hits           5148     5148           
  Misses          540      540           
  Partials         79       79           

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 a95431c...38c4dff. Read the comment docs.

@bednar bednar requested a review from rhajek August 5, 2021 07:08
@bednar bednar marked this pull request as ready for review August 5, 2021 07:08
@bednar bednar merged commit b1dadbd into master Aug 6, 2021
@bednar bednar deleted the fix/streaming-query branch August 6, 2021 11:42
@bednar bednar added this to the 3.2.0 milestone Aug 6, 2021
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.

Memory consumption when retrieving huge result data

3 participants