Skip to content

Conversation

@tanmaykm
Copy link
Member

@tanmaykm tanmaykm commented Apr 5, 2021

gRPCController now has additional parameters to limit both send and receive message sizes,
the default being 4MB. A gRPCMessageTooLargeException is thrown if that is exceeded.

gRPCException is now an abstact type with gRPCMessageTooLargeException and
gRPCServiceCallException as the two concrete implementations.

@codecov-io
Copy link

codecov-io commented Apr 5, 2021

Codecov Report

Merging #6 (49242ab) into main (d28400d) will increase coverage by 0.89%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main       #6      +/-   ##
==========================================
+ Coverage   95.78%   96.68%   +0.89%     
==========================================
  Files           4        5       +1     
  Lines         190      241      +51     
==========================================
+ Hits          182      233      +51     
  Misses          8        8              
Impacted Files Coverage Δ
src/gRPCClient.jl 100.00% <ø> (ø)
src/curl.jl 96.19% <100.00%> (+1.25%) ⬆️
src/grpc.jl 100.00% <100.00%> (ø)
src/limitio.jl 100.00% <100.00%> (ø)

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 d28400d...49242ab. Read the comment docs.

`gRPCController` now has additional parameters to limit both send and receive message sizes,
the default being 4MB. A `gRPCMessageTooLargeException` is thrown if that is exceeded.

`gRPCException` is now an abstact type with `gRPCMessageTooLargeException` and
`gRPCServiceCallException` as the two concrete implementations.
@tanmaykm tanmaykm merged commit 556ae23 into main Apr 5, 2021
@delete-merged-branch delete-merged-branch bot deleted the tan/grpc branch April 5, 2021 09:17
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.

3 participants