[#1090] refactor: Refactor the reader code with builder pattern

[summaryzb]

What changes were proposed in this pull request?

As the title

Why are the changes needed?

#1090

Does this PR introduce any user-facing change?

No.

How was this patch tested?

unit test

[codecov-commenter]

Codecov Report

Merging #1232 (8e29447) into master (77092e4) will increase coverage by 1.23%.
The diff coverage is 54.77%.

@@             Coverage Diff              @@
##             master    #1232      +/-   ##
============================================
+ Coverage     53.57%   54.80%   +1.23%     
- Complexity     2613     2615       +2     
============================================
  Files           395      374      -21     
  Lines         22648    20339    -2309     
  Branches       1933     1937       +4     
============================================
- Hits          12133    11147     -986     
+ Misses         9774     8521    -1253     
+ Partials        741      671      -70     

Files	Coverage Δ
...e/uniffle/client/factory/ShuffleClientFactory.java	93.26% <98.27%> (+27.69%)	⬆️
...pache/hadoop/mapreduce/task/reduce/RssShuffle.java	0.00% <0.00%> (ø)
...library/common/shuffle/impl/RssTezFetcherTask.java	0.00% <0.00%> (ø)
...on/shuffle/orderedgrouped/RssShuffleScheduler.java	46.17% <0.00%> (-0.62%)	⬇️
...che/uniffle/client/impl/ShuffleReadClientImpl.java	76.71% <56.86%> (-1.65%)	⬇️

... and 24 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[summaryzb]

@jerqi PTAL

[jerqi]

LGTM, thanks @summaryzb