Skip to content

Conversation

@10110346
Copy link
Contributor

@10110346 10110346 commented May 31, 2017

What changes were proposed in this pull request?

This PR is to backport #18103 to Spark 2.2

How was this patch tested?

unit test

@ueshin
Copy link
Member

ueshin commented May 31, 2017

ok to test

@SparkQA
Copy link

SparkQA commented May 31, 2017

Test build #77573 has started for PR 18155 at commit 5ab4a96.

@ueshin
Copy link
Member

ueshin commented May 31, 2017

Jenkins, retest this please.

@SparkQA
Copy link

SparkQA commented May 31, 2017

Test build #77579 has finished for PR 18155 at commit 6fc66e8.

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

@ueshin
Copy link
Member

ueshin commented May 31, 2017

LGTM, cc @gatorsmile

@SparkQA
Copy link

SparkQA commented May 31, 2017

Test build #77580 has finished for PR 18155 at commit 6fc66e8.

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

Copy link
Member

@gatorsmile gatorsmile left a comment

Choose a reason for hiding this comment

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

LGTM too

asfgit pushed a commit that referenced this pull request May 31, 2017
… for ceil or floor,the result is not we expected

## What changes were proposed in this pull request?

This PR is to backport #18103 to Spark 2.2

## How was this patch tested?
unit test

Author: liuxian <[email protected]>

Closes #18155 from 10110346/wip-lx-0531.
@gatorsmile
Copy link
Member

Thanks! Merging to 2.0.

Could you manually close this PR? Thanks!

@10110346
Copy link
Contributor Author

10110346 commented Jun 1, 2017

@gatorsmile OK,thanks

@10110346 10110346 closed this Jun 1, 2017
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