Revert "[Improvement] Skip blocks when read from memory (#294)"

[xianjingfeng]

This reverts commit 55191c4.

What changes were proposed in this pull request?

Revert #294

Why are the changes needed?

BlockId is discontinuous, so BLOCKID_RANGE is not a good choice to filter memory data

Does this PR introduce any user-facing change?

No

How was this patch tested?

No need

[codecov-commenter]

Codecov Report

Merging #403 (9668101) into master (ddf8384) will increase coverage by 0.05%.
The diff coverage is 18.75%.

@@             Coverage Diff              @@
##             master     #403      +/-   ##
============================================
+ Coverage     58.51%   58.56%   +0.05%     
+ Complexity     1616     1604      -12     
============================================
  Files           195      194       -1     
  Lines         11069    10984      -85     
  Branches        977      961      -16     
============================================
- Hits           6477     6433      -44     
+ Misses         4209     4174      -35     
+ Partials        383      377       -6     

Impacted Files	Coverage Δ
.../java/org/apache/hadoop/mapreduce/RssMRConfig.java	23.07% <ø> (ø)
...pache/hadoop/mapreduce/task/reduce/RssShuffle.java	0.00% <ø> (ø)
.../java/org/apache/spark/shuffle/RssSparkConfig.java	93.75% <ø> (-0.13%)	⬇️
...e/uniffle/client/factory/ShuffleClientFactory.java	0.00% <0.00%> (ø)
...client/request/CreateShuffleReadClientRequest.java	0.00% <0.00%> (ø)
...rg/apache/uniffle/client/util/RssClientConfig.java	0.00% <ø> (ø)
.../java/org/apache/uniffle/common/util/RssUtils.java	60.58% <ø> (-7.00%)	⬇️
...pache/uniffle/server/ShuffleServerGrpcService.java	0.81% <ø> (+<0.01%)	⬆️
.../org/apache/uniffle/server/ShuffleTaskManager.java	77.22% <0.00%> (ø)
...uniffle/storage/factory/ShuffleHandlerFactory.java	0.00% <0.00%> (ø)
... and 7 more

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

[jerqi]

LGTM, thanks @xianjingfeng