BUG FIX: #3420

[yiding-he]

1. IOUtils.write(InputStream, OutputStream, int) BUG DO NOT USE org.apache.dubbo.common.utils.IOUtils.write() for your own project #3420
2. IOUtils.write(Reader, Writer, int) not using bufferSize parameter.

org.apache.dubbo.common.io.Bytes#unzip() is now using fixed write() method so it is worth running a BytesTest#testZip() test.

[codecov-io]

Codecov Report

Merging #3427 into master will increase coverage by 0.06%.
The diff coverage is 88.88%.

@@             Coverage Diff              @@
##             master    #3427      +/-   ##
============================================
+ Coverage     63.89%   63.96%   +0.06%     
  Complexity       71       71              
============================================
  Files           662      662              
  Lines         28535    28569      +34     
  Branches       4758     4784      +26     
============================================
+ Hits          18233    18274      +41     
+ Misses         8092     8083       -9     
- Partials       2210     2212       +2

Impacted Files	Coverage Δ	Complexity Δ
...in/java/org/apache/dubbo/common/utils/IOUtils.java	88.88% <88.88%> (-0.25%)	0 <0> (ø)
.../apache/dubbo/remoting/transport/AbstractPeer.java	58.69% <0%> (-4.35%)	0% <0%> (ø)
...dubbo/common/status/support/LoadStatusChecker.java	46.15% <0%> (-3.85%)	0% <0%> (ø)
...c/main/java/org/apache/dubbo/rpc/RpcException.java	82.75% <0%> (-3.45%)	0% <0%> (ø)
...ting/exchange/support/header/HeartbeatHandler.java	84.44% <0%> (-1.92%)	0% <0%> (ø)
.../remoting/transport/netty4/NettyClientHandler.java	68.42% <0%> (-1.23%)	0% <0%> (ø)
...e/dubbo/remoting/transport/netty4/NettyClient.java	74.6% <0%> (-1.21%)	0% <0%> (ø)
...he/dubbo/remoting/transport/netty/NettyClient.java	66.1% <0%> (-1.14%)	0% <0%> (ø)
...ache/dubbo/remoting/transport/mina/MinaClient.java	58.46% <0%> (-0.92%)	0% <0%> (ø)
.../main/java/org/apache/dubbo/qos/textui/TTable.java	83.79% <0%> (-0.87%)	0% <0%> (ø)
... and 19 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 c2c9de9...219e1c7. Read the comment docs.

[tswstarplanet]

I think your pr may have some problems. You don’t have to delete all the content of the origin file and then add all content including your fix to the file.
Please check your pr and resubmit it.

[yiding-he]

@tswstarplanet It's just something like \n -> \r\n. open "Diff Settings" and check "Hide whitespace changes", you will see actual changes. I'll be careful next time.

[kun-song]

Hi @yiding-he nice work, but please make sure you have autocrlf = false in your git configuration since we don't use \n\r for a newline, thanks :)

[ralf0131]

This pull request is difficult to review. Please only keep the minimal change. Why do you change all the lines?