[SPARK-3690] Closing shuffle writers we swallow more important exception

[epahomov]

No description provided.

[AmplabJenkins]

Can one of the admins verify this patch?

[JoshRosen]

Jenkins, this is ok to test.

[SparkQA]

QA tests have started for PR 2537 at commit a0b7de4.

• This patch merges cleanly.

[SparkQA]

QA tests have finished for PR 2537 at commit a0b7de4.

• This patch fails unit tests.
• This patch merges cleanly.
• This patch adds no public classes.

[AmplabJenkins]

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/20812/

[SparkQA]

QA tests have started for PR 2537 at commit a0b7de4.

• This patch merges cleanly.

[SparkQA]

QA tests have finished for PR 2537 at commit a0b7de4.

• This patch passes unit tests.
• This patch merges cleanly.
• This patch adds no public classes.

[JoshRosen]

Good catch. This looks good to me, so I'm going to merge it. Thanks!