[SPARK-22211][SQL][FOLLOWUP] Fix bad merge for tests

[henryr]

What changes were proposed in this pull request?

The merge of SPARK-22211 to branch-2.2 dropped a couple of important lines that made sure the tests that compared plans did so with both plans having been analyzed. Fix by reintroducing the correct analysis statements.

How was this patch tested?

Re-ran LimitPushdownSuite. All tests passed.

[dongjoon-hyun]

+1, LGTM.
Thank you, @henryr .

[SparkQA]

Test build #83615 has finished for PR 19701 at commit 890f608.

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

[dongjoon-hyun]

The failure is not related to this.

Traceback (most recent call last):
  File "/home/anaconda/lib/python2.7/runpy.py", line 162, in _run_module_as_main
    "__main__", fname, loader, pkg_name)
  File "/home/anaconda/lib/python2.7/runpy.py", line 72, in _run_code
    exec code in run_globals
  File "/home/jenkins/workspace/SparkPullRequestBuilder/python/pyspark/sql/tests.py", line 196, in <module>
    class SQLTests(ReusedPySparkTestCase):
  File "/home/jenkins/workspace/SparkPullRequestBuilder/python/pyspark/sql/tests.py", line 2200, in SQLTests
    @unittest.skipIf(not _have_pandas, "Pandas not installed")
NameError: name '_have_pandas' is not defined

[dongjoon-hyun]

Retest this please.

[SparkQA]

Test build #83624 has finished for PR 19701 at commit 890f608.

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

[gatorsmile]

Retest this please.

[gatorsmile]

LGTM

[gatorsmile]

The test cases already passed.

Thanks! Merged to 2.2.

[dongjoon-hyun]

Thank you, @gatorsmile and @henryr !

[dongjoon-hyun]

Please close this PR, @henryr . branch-2.2 PR is not closed automatically.

[felixcheung]

thanks!