[SPARK-20633][SQL] FileFormatWriter wrap the FetchFailedException which breaks job's failover

[lshmouse]

What changes were proposed in this pull request?

Handle the fetch failed exception separately in FileFormatWriter.

How was this patch tested?

manual tests

Please review http://spark.apache.org/contributing.html before opening a pull request.

[lshmouse]

Pending. I am formatting the pull name~

[mridulm]

This is not required anymore - post @squito's change.
Though it cant hurt I guess.

[squito]

yeah, this isn't a bug anymore, but lets fix it -- cleaner error msg, and at least it will be easier for folks to see the fix in the future, as multiple people have stumbled on it and not realized it was fixed.

@lshmouse can you update the description to reference the fix in SPARK-19276?

[squito]

Jenkins, ok to test

[SparkQA]

Test build #76579 has finished for PR 17893 at commit c869d9c.

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

[squito]

Jenkins, retest this please

[SparkQA]

Test build #76590 has finished for PR 17893 at commit c869d9c.

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

[felixcheung]

Please rebase to pick up fix for R tests

[squito]

Jenkins, retest this please

[SparkQA]

Test build #76689 has finished for PR 17893 at commit c869d9c.

• This patch fails from timeout after a configured wait of `250m`.
• This patch merges cleanly.
• This patch adds no public classes.

[squito]

since it seems @lshmouse is not updating this, I've resubmitted this here #18145, but all credit to the original author.