You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@datafusion.apache.org by "alamb (via GitHub)" <gi...@apache.org> on 2025/10/14 21:06:43 UTC
[I] CI is failing on main [datafusion]
alamb opened a new issue, #18062:
URL: https://github.com/apache/datafusion/issues/18062
### Describe the bug
An example failure:
https://github.com/apache/datafusion/actions/runs/18508018377/job/52742095719
```
running 1 test
test engines::conversion::tests::test_big_decimal_to_str ... ok
test result: ok. 1 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.00s
Running bin/sqllogictests.rs (target/ci/deps/sqllogictests-8bd1cd21eda5b184)
External error: 1 errors in file /__w/datafusion/datafusion/datafusion/sqllogictest/test_files/current_date_timezone.slt
1. query result mismatch:
[SQL] SELECT current_date() = cast(now() as date);
[Diff] (-expected|+actual)
- true
+ false
at /__w/datafusion/datafusion/datafusion/sqllogictest/test_files/current_date_timezone.slt:44
Error: Execution("1 failures\n\nbacktrace: 0: sqllogictests::run_tests::{{closure}}\n 1: tokio::runtime::park::CachedParkThread::block_on::{{closure}}\n 2: tokio::runtime::park::CachedParkThread::block_on\n 3: tokio::runtime::context::blocking::BlockingRegionGuard::block_on\n 4: tokio::runtime::scheduler::multi_thread::MultiThread::block_on::{{closure}}\n 5: tokio::runtime::context::runtime::enter_runtime\n 6: tokio::runtime::scheduler::multi_thread::MultiThread::block_on\n 7: tokio::runtime::runtime::Runtime::block_on_inner\n 8: tokio::runtime::runtime::Runtime::block_on\n 9: sqllogictests::main\n 10: core::ops::function::FnOnce::call_once\n 11: std::sys::backtrace::__rust_begin_short_backtrace\n 12: std::rt::lang_start::{{closure}}\n 13: std::rt::lang_start_internal\n 14: std::rt::lang_start\n 15: main\n 16: <unknown>\n 17: __libc_start_main\n 18: _start\n")
error: test failed, to rerun pass `-p datafusion-sqllogictest --test sqllogictests`
```
### To Reproduce
_No response_
### Expected behavior
I expect CI to be clean
### Additional context
<img width="1582" height="787" alt="Image" src="https://github.com/user-attachments/assets/a3af1d74-dcd6-4323-807d-aa89f8df8679" />
The test was introduced (but passed) in:
- https://github.com/apache/datafusion/pull/18034
It first started failing in the next commit
- https://github.com/apache/datafusion/pull/18049
So I am somewhat confused
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For additional commands, e-mail: github-help@datafusion.apache.org
Re: [I] CI is failing on main / `current_date() = cast(now() as date);` [datafusion]
Posted by "dqkqd (via GitHub)" <gi...@apache.org>.
dqkqd commented on issue #18062:
URL: https://github.com/apache/datafusion/issues/18062#issuecomment-3403941785
@comphead
Yeah I think we can comment the test.
I found that @Weijun-H is working on `now` in #18017.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For additional commands, e-mail: github-help@datafusion.apache.org
Re: [I] CI is failing on main / `current_date() = cast(now() as date);` [datafusion]
Posted by "dqkqd (via GitHub)" <gi...@apache.org>.
dqkqd commented on issue #18062:
URL: https://github.com/apache/datafusion/issues/18062#issuecomment-3403649357
I run `datafusion-cli` on main.
`current_date` returns expected behavior as introduced in the PR. It changes based on timezone.
```
> SET datafusion.execution.time_zone = '+00:00';
0 row(s) fetched.
Elapsed 0.001 seconds.
> select current_date();
+----------------+
| current_date() |
+----------------+
| 2025-10-14 |
+----------------+
1 row(s) fetched.
Elapsed 0.004 seconds.
> SET datafusion.execution.time_zone = '+05:00';
0 row(s) fetched.
Elapsed 0.001 seconds.
> select current_date();
+----------------+
| current_date() |
+----------------+
| 2025-10-15 |
+----------------+
1 row(s) fetched.
Elapsed 0.004 seconds.
```
But `now` doesn't.
```
DataFusion CLI v50.2.0
> SET datafusion.execution.time_zone = '+00:00';
0 row(s) fetched.
Elapsed 0.003 seconds.
> select now();
+--------------------------------+
| now() |
+--------------------------------+
| 2025-10-14T21:16:57.781868096Z |
+--------------------------------+
1 row(s) fetched.
Elapsed 0.005 seconds.
> SET datafusion.execution.time_zone = '+05:00';
0 row(s) fetched.
Elapsed 0.001 seconds.
> select now();
+--------------------------------+
| now() |
+--------------------------------+
| 2025-10-14T21:17:02.014016475Z |
+--------------------------------+
1 row(s) fetched.
Elapsed 0.003 seconds.
>
```
Might require #17993 to make the test work.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For additional commands, e-mail: github-help@datafusion.apache.org
Re: [I] CI is failing on main / `current_date() = cast(now() as date);` [datafusion]
Posted by "comphead (via GitHub)" <gi...@apache.org>.
comphead commented on issue #18062:
URL: https://github.com/apache/datafusion/issues/18062#issuecomment-3403864208
@dqkqd can we comment out test until you get the permanent fix with time merged?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For additional commands, e-mail: github-help@datafusion.apache.org
Re: [I] CI is failing on main / `current_date() = cast(now() as date);` [datafusion]
Posted by "Jefffrey (via GitHub)" <gi...@apache.org>.
Jefffrey closed issue #18062: CI is failing on main / `current_date() = cast(now() as date);`
URL: https://github.com/apache/datafusion/issues/18062
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For additional commands, e-mail: github-help@datafusion.apache.org