[SPARK-39080][SHUFFLE]Optimize shuffle error handler

[wankunde]

What changes were proposed in this pull request?

Optimize ErrorHandler code.

Why are the changes needed?

Shuffle ErrorHandler only has two stateless method, so we do not need create a ErrorHandler instance for each shuffle fetcher or shuffle pusher.

Does this PR introduce any user-facing change?

No

How was this patch tested?

Exists UT

[wankunde]

As a followup of SPARK-38965, if there are no BLOCK_APPEND_COLLISION exception, shuffle fetcher and shuffle pusher do not handle the exception returned from server, and RetryingBlockTransferor will retry if there is a IOException and errorHandler.shouldRetryError(e) is true.

[AmplabJenkins]

Can one of the admins verify this patch?

[mridulm]

I have not looked at the PR in detail, but we have to be careful to preserve backwards compatible and not make breaking changes.

[github-actions[bot]]

We're closing this PR because it hasn't been updated in a while. This isn't a judgement on the merit of the PR in any way. It's just a way of keeping the PR queue manageable. If you'd like to revive this PR, please reopen it and ask a committer to remove the Stale tag!