Skip to content

Conversation

@ueshin
Copy link
Member

@ueshin ueshin commented Sep 17, 2018

What changes were proposed in this pull request?

There are some mistakes in examples of newly added functions. Also the format of the example results are not unified. We should fix them.

How was this patch tested?

Manually executed the examples.

@ueshin
Copy link
Member Author

ueshin commented Sep 17, 2018

cc @dongjoon-hyun @gatorsmile

Copy link
Member

@HyukjinKwon HyukjinKwon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@SparkQA
Copy link

SparkQA commented Sep 17, 2018

Test build #96118 has finished for PR 22437 at commit 1ae77da.

  • This patch fails due to an unknown error code, -9.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Sep 17, 2018

Test build #96117 has finished for PR 22437 at commit 865b09b.

  • This patch fails due to an unknown error code, -9.
  • This patch merges cleanly.
  • This patch adds no public classes.

@ueshin
Copy link
Member Author

ueshin commented Sep 17, 2018

Jenkins, retest this please.

@SparkQA
Copy link

SparkQA commented Sep 17, 2018

Test build #96122 has finished for PR 22437 at commit 1ae77da.

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

@ueshin
Copy link
Member Author

ueshin commented Sep 17, 2018

Jenkins, retest this please.

Copy link
Member

@dongjoon-hyun dongjoon-hyun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1, LGTM.

I also did double-check manually. Thank you, @ueshin .

@SparkQA
Copy link

SparkQA commented Sep 17, 2018

Test build #96127 has finished for PR 22437 at commit 1ae77da.

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

@HyukjinKwon
Copy link
Member

Merged to master and branch-2.4.

asfgit pushed a commit that referenced this pull request Sep 17, 2018
…sults.

## What changes were proposed in this pull request?

There are some mistakes in examples of newly added functions. Also the format of the example results are not unified. We should fix them.

## How was this patch tested?

Manually executed the examples.

Closes #22437 from ueshin/issues/SPARK-25431/fix_examples_2.

Authored-by: Takuya UESHIN <[email protected]>
Signed-off-by: hyukjinkwon <[email protected]>
(cherry picked from commit 8cf6fd1)
Signed-off-by: hyukjinkwon <[email protected]>
@asfgit asfgit closed this in 8cf6fd1 Sep 17, 2018
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