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

test case timestamp_add_interval_months failed on master branch #3321

Closed
waynexia opened this issue Sep 1, 2022 · 4 comments · Fixed by #3337
Closed

test case timestamp_add_interval_months failed on master branch #3321

waynexia opened this issue Sep 1, 2022 · 4 comments · Fixed by #3337
Labels
bug Something isn't working help wanted Extra attention is needed

Comments

@waynexia
Copy link
Member

waynexia commented Sep 1, 2022

Describe the bug

UT case timestamp_add_interval_months failed to run on master (commit 3d37016)

To Reproduce

git checkout 3d37016de3647d90e6f78fd0e106142923799969
cargo test

I got this error:

thread 'sql::timestamp::timestamp_add_interval_months' panicked at 'called `Option::unwrap()` on a `None` value', datafusion/core/tests/sql/timestamp.rs:1414:58

Expected behavior

Pass this test

Additional context
Add any other context about the problem here.

@waynexia waynexia added the bug Something isn't working label Sep 1, 2022
@waynexia
Copy link
Member Author

waynexia commented Sep 1, 2022

I propose to add this trigger to our CI file that allows us to trigger an action manually

@waynexia
Copy link
Member Author

waynexia commented Sep 1, 2022

It looks like both #3305 (action) and #3320 (action) are affected by this.

@waynexia
Copy link
Member Author

waynexia commented Sep 1, 2022

I propose to add this trigger to our CI file that allows us to trigger an action manually

Submitted #3323

@alamb
Copy link
Contributor

alamb commented Sep 1, 2022

We have disabled the failing test in #3329

It would be great if someone could help fix the test for real

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants