Skip to content

Add RPCHttpClient to handle RPCException at client side#2165

Merged
xerial merged 16 commits intomasterfrom
rpc-client
May 10, 2022
Merged

Add RPCHttpClient to handle RPCException at client side#2165
xerial merged 16 commits intomasterfrom
rpc-client

Conversation

@xerial
Copy link
Copy Markdown
Member

@xerial xerial commented May 9, 2022

  • Use application/msgpack
  • Add RPCHttpClient base implementation
  • Read RPC error response from the http response body
  • Throw RPCException upon errors
  • Async RPC client
  • Scala.js client
  • Add RPC specific parameters to HttpClientConfig

I'll change the http client generator in a separate PR so as not to mixup adding RPCClient and its migration

@codecov
Copy link
Copy Markdown

codecov Bot commented May 9, 2022

Codecov Report

Merging #2165 (1f1c0c3) into master (a9a04eb) will decrease coverage by 0.10%.
The diff coverage is 50.70%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2165      +/-   ##
==========================================
- Coverage   82.66%   82.55%   -0.11%     
==========================================
  Files         307      307              
  Lines       12107    12157      +50     
  Branches      774      789      +15     
==========================================
+ Hits        10008    10036      +28     
- Misses       2099     2121      +22     
Impacted Files Coverage Δ
...rc/main/scala/wvlet/airframe/http/HttpHeader.scala 0.00% <0.00%> (ø)
...la/wvlet/airframe/http/HttpResponseBodyCodec.scala 81.81% <ø> (ø)
...n/scala/wvlet/airframe/http/HttpClientConfig.scala 25.00% <14.28%> (-15.00%) ⬇️
...frame/http/client/URLConnectionClientBackend.scala 70.00% <25.00%> (-30.00%) ⬇️
...main/scala/wvlet/airframe/http/RPCHttpClient.scala 37.14% <37.14%> (ø)
...http/src/main/scala/wvlet/airframe/http/Http.scala 76.78% <50.00%> (+1.31%) ⬆️
...la/wvlet/airframe/http/finagle/FinagleClient.scala 92.13% <100.00%> (ø)
...la/wvlet/airframe/http/finagle/FinagleServer.scala 72.22% <100.00%> (ø)
...cala/wvlet/airframe/http/okhttp/OkHttpClient.scala 92.98% <100.00%> (ø)
...rc/main/scala/wvlet/airframe/http/HttpClient.scala 89.13% <100.00%> (ø)
... and 6 more

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 a9a04eb...1f1c0c3. Read the comment docs.

@xerial xerial changed the title Handle RPCException at clients Add RPCHttpClient to handle RPCException at client side May 10, 2022
@xerial xerial added this to the Airframe 22 milestone May 10, 2022
@xerial xerial merged commit 782b343 into master May 10, 2022
@xerial xerial deleted the rpc-client branch June 2, 2022 16:22
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.

1 participant