Skip to content

Conversation

fakegermano
Copy link

The python spark datasets example for the mapreduce was returning error, when using .select().as() function.
Replaces the error to use the .select().name() function, that does the same thing and works

What changes were proposed in this pull request?

no changes

How was this patch tested?

no significant changes were made to the spark code. Just documentation changes. The change I implemented was tested manually over spark 2.2.0

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

The python spark datasets example for the mapreduce was returning error, when using .select().as() function.
Replaces the error to use the .select().name() function, that does the same thing and works
@AmplabJenkins
Copy link

Can one of the admins verify this patch?

@srowen
Copy link
Member

srowen commented Sep 19, 2017

Have a look at http://spark.apache.org/contributing.html -- maybe prefix the title with [MINOR][DOCS] for completeness?

Are there other instances of this same issue in the Pyspark docs?

asfgit pushed a commit that referenced this pull request Sep 25, 2017
## What changes were proposed in this pull request?

Updated docs so that a line of python in the quick start guide executes. Closes #19283

## How was this patch tested?

Existing tests.

Author: John O'Leary <[email protected]>

Closes #19326 from jgoleary/issues/22107.

(cherry picked from commit 20adf9a)
Signed-off-by: hyukjinkwon <[email protected]>
@asfgit asfgit closed this in 20adf9a Sep 25, 2017
MatthewRBruce pushed a commit to Shopify/spark that referenced this pull request Jul 31, 2018
## What changes were proposed in this pull request?

Updated docs so that a line of python in the quick start guide executes. Closes apache#19283

## How was this patch tested?

Existing tests.

Author: John O'Leary <[email protected]>

Closes apache#19326 from jgoleary/issues/22107.

(cherry picked from commit 20adf9a)
Signed-off-by: hyukjinkwon <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants