[Bug] Fix to use TezIdHelper to getTaskAttemptId instead of default IdHelper.

[lifeSo]

Code of Conduct

• I agree to follow this project's Code of Conduct

Search before asking

• I have searched in the issues and found no similar issues.

Describe the bug

when there is no task fail and attempt, the default IdHelper code before will work ok, but if where is task fail and attempt, it will filter block id.

The filter code:

• ShuffleReadClientImp.java

    List<Long> removeBlockIds = Lists.newArrayList();
    blockIdBitmap.forEach(
        bid -> {
          if (!taskIdBitmap.contains(idHelper.getTaskAttemptId(bid))) {
            removeBlockIds.add(bid);
          }
        });

    for (long rid : removeBlockIds) {
      blockIdBitmap.removeLong(rid);
    }

idHelper.getTaskAttemptId(bid) will return wrong task attemt id if there is task attempt and use default idHelper.

Affects Version(s)

master

Uniffle Server Log Output

No response

Uniffle Engine Log Output

No response

Uniffle Server Configurations

No response

Uniffle Engine Configurations

No response

Additional context

No response

Are you willing to submit PR?

• Yes I am willing to submit a PR!