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

Fix stream creation concurrency in new_stream #66

Merged
merged 1 commit into from
Jun 7, 2022

Conversation

psalin
Copy link
Collaborator

@psalin psalin commented Dec 30, 2021

The stream concurrency fix for joedevivo/chatterbox#136 had only been taken into use in
grpcbox_client_stream:send_request. This commit adds the fix also
to grpcbox_client_stream:new_stream.

@codecov
Copy link

codecov bot commented Dec 30, 2021

Codecov Report

Merging #66 (e4fd229) into main (0166760) will decrease coverage by 0.02%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #66      +/-   ##
==========================================
- Coverage   38.99%   38.96%   -0.03%     
==========================================
  Files          28       28              
  Lines        2090     2089       -1     
==========================================
- Hits          815      814       -1     
  Misses       1275     1275              
Impacted Files Coverage Δ
src/grpcbox_client_stream.erl 68.91% <ø> (-0.42%) ⬇️

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 0166760...e4fd229. Read the comment docs.

The stream concurrency fix had only been taken into use in
grpcbox_client_stream:send_request. This commit adds the fix also
to grpcbox_client_stream:new_stream.
@psalin psalin force-pushed the fix_stream_creation_concurrency branch from 4422af8 to e4fd229 Compare February 7, 2022 11:55
@tsloughter tsloughter merged commit fd5c63b into tsloughter:main Jun 7, 2022
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