Deprecate CircuitBreakerRpcClient.newPerHostAndMethodDecorator()

[KarboniteKream]

Motivation

• Expanding CircuitBreakerMapping to include request path as a key #3135 deprecated newPerHostAndMethodDecorator(), but the method in CircuitBreakerRpcClient hasn't been marked as deprecated

Modifications

• Deprecate CircuitBreakerRpcClient.newPerHostAndMethodDecorator()
• Update wrong PR link in release notes
• Update documentation

Result

• Method deprecations are consistent

[minwoox]

Thanks a lot, @KarboniteKream!

[codecov]

Codecov Report

Merging #3249 (bf6907f) into master (ccff172) will increase coverage by 0.01%.
The diff coverage is 86.95%.

@@             Coverage Diff              @@
##             master    #3249      +/-   ##
============================================
+ Coverage     74.05%   74.07%   +0.01%     
- Complexity    12999    13000       +1     
============================================
  Files          1133     1134       +1     
  Lines         49197    49213      +16     
  Branches       6259     6255       -4     
============================================
+ Hits          36435    36456      +21     
- Misses         9546     9551       +5     
+ Partials       3216     3206      -10     

Impacted Files	Coverage Δ	Complexity Δ
...ia/client/circuitbreaker/CircuitBreakerClient.java	83.07% <ø> (ø)	26.00 <0.00> (ø)
...client/circuitbreaker/CircuitBreakerRpcClient.java	80.95% <ø> (ø)	10.00 <0.00> (ø)
...spring/ArmeriaServerGracefulShutdownLifecycle.java	70.00% <70.00%> (ø)	5.00 <5.00> (?)
...dpoint/healthcheck/HealthCheckedEndpointGroup.java	82.71% <100.00%> (+0.55%)	22.00 <0.00> (+1.00)
...p/armeria/common/metric/MeterIdPrefixFunction.java	94.11% <100.00%> (+0.78%)	7.00 <1.00> (+1.00)
...meria/spring/AbstractArmeriaAutoConfiguration.java	75.00% <100.00%> (+0.58%)	13.00 <1.00> (+1.00)
...p/armeria/client/endpoint/OrElseEndpointGroup.java	54.28% <0.00%> (-14.29%)	8.00% <0.00%> (-3.00%)
...om/linecorp/armeria/client/HttpSessionHandler.java	71.59% <0.00%> (-9.47%)	49.00% <0.00%> (-5.00%)
...rp/armeria/common/stream/StreamMessageWrapper.java	86.95% <0.00%> (-3.96%)	11.00% <0.00%> (ø%)
...eria/internal/common/DefaultSplitHttpResponse.java	77.38% <0.00%> (-2.86%)	14.00% <0.00%> (ø%)
... and 53 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 ea4e7f5...bf6907f. Read the comment docs.

[ikhoon]

Thanks! @KarboniteKream 🙇‍♂️

[trustin]

Thanks a lot, @KarboniteKream! No more broken windows 🙇