[SPARK-11329] [SQL] Cleanup from spark-11329 fix.

[nongli]

No description provided.

[srowen]

Dumb question, but why wasn't this part of SPARK-11329? or attached to it?

[nongli]

My bad. Does this do the trick?

[yhuai]

@srowen Since the original pr did not have any fundamental issue and we will probably see a burst of commits (as we will cut the branch soon), I committed the original one to avoid potential merge conflict. We will attach this pr to the original jira. So, it will be clear from the commit history.

[srowen]

I figured. My motivation is mostly to make sure that it's easy to find all the commits that belong to one logical change. And maybe reducing noise but that's minor.

[yhuai]

@srowen yeah. That's a good point.

[SparkQA]

Test build #44945 has finished for PR 9442 at commit a53bad4.

• This patch passes all tests.
• This patch merges cleanly.
• This patch adds the following public classes (experimental):\n * case class ExecutorLostFailure(\n * case class DecimalLit(chars: String) extends Token\n

[yhuai]

Thanks! LGTM. Merging to master!