Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[SPARK-20763][SQL][Backport-2.1] The function of month and day return the value which is not we expected. #18054

Closed
wants to merge 1 commit into from

Conversation

10110346
Copy link
Contributor

What changes were proposed in this pull request?

This PR is to backport #17997 to Spark 2.1

when the date before "1582-10-04", the function of month and day return the value which is not we expected.
How was this patch tested?

unit tests

@ueshin
Copy link
Member

ueshin commented May 22, 2017

Jenkins, test this please.

@hvanhovell
Copy link
Contributor

ok to test

@SparkQA
Copy link

SparkQA commented May 22, 2017

Test build #77170 has started for PR 18054 at commit 9f248f1.

@ueshin
Copy link
Member

ueshin commented May 22, 2017

Jenkins, retest this please.

@SparkQA
Copy link

SparkQA commented May 22, 2017

Test build #77172 has finished for PR 18054 at commit 9f248f1.

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

@ueshin
Copy link
Member

ueshin commented May 23, 2017

LGTM

@ueshin
Copy link
Member

ueshin commented May 23, 2017

Thanks, merging to 2.1.

asfgit pushed a commit that referenced this pull request May 23, 2017
…turn the value which is not we expected.

What changes were proposed in this pull request?

This PR is to backport #17997 to Spark 2.1

when the date before "1582-10-04", the function of month and day return the value which is not we expected.
How was this patch tested?

unit tests

Author: liuxian <[email protected]>

Closes #18054 from 10110346/wip-lx-0522.
@ueshin
Copy link
Member

ueshin commented May 23, 2017

Could you close the PR manually?

@10110346 10110346 closed this May 23, 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