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.

Sign up for GitHub

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

Jump to bottom

[SPARK-35088][SQL][FOLLOWUP] Add test case for TimestampNTZ sequence with default step #33462

Closed

beliefer wants to merge 1 commit into apache:master from beliefer:SPARK-36090-followup

Contributor

beliefer commented Jul 21, 2021

What changes were proposed in this pull request?

This PR follows up #33360 and add test case for TimestampNTZ sequence with default step.

Why are the changes needed?

Improve test coverage.

Does this PR introduce any user-facing change?

'No'.
Just add test cases.

How was this patch tested?

New tests.


          Add Sequence with default step

1e16de7

github-actions bot added the SQL label

SparkQA commented Jul 21, 2021

Kubernetes integration test unable to build dist.

exiting with code: 1
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/45924/

SparkQA commented Jul 21, 2021

Test build #141407 has finished for PR 33462 at commit 1e16de7.

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

Contributor Author

beliefer commented Jul 22, 2021

ping @gengliangwang @MaxGekk

gengliangwang approved these changes

View reviewed changes

Member

gengliangwang commented Jul 22, 2021

Thanks, merging to master

gengliangwang closed this in

900b72a

Contributor Author

beliefer commented Jul 23, 2021

@gengliangwang Thanks a lot!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

SQL