Skip to content

Conversation

@ongmingyang
Copy link
Contributor

What changes were proposed in this pull request?

  • Following the first few examples in this file, the remaining methods should also be methods of df.na not df.
  • Filled in some missing parentheses

How was this patch tested?

N/A

## What changes were proposed in this pull request?

* The methods written in this file are methods of `df.na` not `df`.
* Filled in some missing parentheses

## How was this patch tested?

N/A
@SparkQA
Copy link

SparkQA commented Jun 23, 2017

Test build #3807 has finished for PR 18398 at commit 6c6df33.

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

@SparkQA
Copy link

SparkQA commented Jun 23, 2017

Test build #3809 has finished for PR 18398 at commit 6c6df33.

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

@gatorsmile
Copy link
Member

Thanks! Merging to master/2.2/2.1

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

* Following the first few examples in this file, the remaining methods should also be methods of `df.na` not `df`.
* Filled in some missing parentheses

## How was this patch tested?

N/A

Author: Ong Ming Yang <[email protected]>

Closes #18398 from ongmingyang/master.

(cherry picked from commit 4cc6295)
Signed-off-by: Xiao Li <[email protected]>
@asfgit asfgit closed this in 4cc6295 Jun 23, 2017
asfgit pushed a commit that referenced this pull request Jun 23, 2017
## What changes were proposed in this pull request?

* Following the first few examples in this file, the remaining methods should also be methods of `df.na` not `df`.
* Filled in some missing parentheses

## How was this patch tested?

N/A

Author: Ong Ming Yang <[email protected]>

Closes #18398 from ongmingyang/master.

(cherry picked from commit 4cc6295)
Signed-off-by: Xiao Li <[email protected]>
robert3005 pushed a commit to palantir/spark that referenced this pull request Jun 29, 2017
## What changes were proposed in this pull request?

* Following the first few examples in this file, the remaining methods should also be methods of `df.na` not `df`.
* Filled in some missing parentheses

## How was this patch tested?

N/A

Author: Ong Ming Yang <[email protected]>

Closes apache#18398 from ongmingyang/master.
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.

4 participants