[MINOR][PySpark][DOC] Fix wrongly formatted examples in PySpark documentation

[HyukjinKwon]

What changes were proposed in this pull request?

This PR fixes wrongly formatted examples in PySpark documentation as below:

• SparkSession

  • Before

    

  • After

    

• Builder
  • Before
    
  • After
    

This PR also fixes several similar instances across the documentation in sql PySpark module.

How was this patch tested?

N/A

[SparkQA]

Test build #61804 has finished for PR 14063 at commit a8968bf.

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

[HyukjinKwon]

Oh, I will fix them up soon.

[SparkQA]

Test build #61808 has finished for PR 14063 at commit 1cacb89.

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

[SparkQA]

Test build #61810 has finished for PR 14063 at commit 9edca4f.

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

[rxin]

Can you verify the output after the all the follow-up commits?

[HyukjinKwon]

Sure, It seems the output (representation) for nested class is varied by the version of Python.

First, I updated this to <pyspark.sql.session.SparkSession.Builder object at ...> and got this error below with pypy:

Expected:
    <pyspark.sql.session.SparkSession.Builder object at ...>
Got:
    <pyspark.sql.session.Builder object at 0x000000000163e4b8>

then I updated this to <pyspark.sql.session.Builder object at ...> and got this error below with python 3.4:

Expected:
    <pyspark.sql.session.Builder object at ...>
Got:
    <pyspark.sql.session.SparkSession.Builder object at 0x7f8c5657d9e8>

I tested this with Python 2.7 and 3.4 with this module below:

• test.py

  class SparkSession(object):
      class Builder(object):
          pass
      builder = Builder()

• Python 2.7 shell

  >>> from test import SparkSession
  >>> SparkSession.builder
  <test.Builder object at 0x7fa9224fa650>

• Python 3.4 shell

  >>> from test import SparkSession
  >>> SparkSession.builder
  <test.SparkSession.Builder object at 0x7f11e79e18d0>

[rxin]

Output is fine -- I just want to make sure the doc is correct.

[HyukjinKwon]

Ah, yes it seems fine.

[rxin]

Can you scan the Python docs and see if there are more instances?

http://people.apache.org/~pwendell/spark-releases/spark-2.0.0-rc2-docs/api/python/pyspark.sql.html

I found at least one ...

[HyukjinKwon]

Sure!

[HyukjinKwon]

I just double-checked the built documentation after the last commit. Do you want me to scan other modules (maybe in another PR) as well?

[rxin]

If you can please check other modules too. Thanks!

[HyukjinKwon]

Sure! Thanks.

[SparkQA]

Test build #61825 has finished for PR 14063 at commit d0672ae.

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

[SparkQA]

Test build #61826 has finished for PR 14063 at commit 607e442.

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

[HyukjinKwon]

@rxin It seems these are all instances related with this (I can't believe!). I just simply corrected indentation for a doc-test in mllib.

[SparkQA]

Test build #61833 has finished for PR 14063 at commit 673150d.

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

[rxin]

Merging in master/2.0. Thanks.