-
Notifications
You must be signed in to change notification settings - Fork 50
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
Tracking: run SQLite sqllogictest suite #148
Comments
🤣🤣 |
Another thing that'll break those tests is lack of support for |
Another is that sqlite tests think This one is more for the runner, not the harness. Or for anyone trying to repurpose SQLite tests for other backends. |
And #122 (trailing comments) |
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
Why does that require sqllogictest’s support? 👀
…On Sat, 9 Dec 2023 at 06:15, Tv ***@***.***> wrote:
SQLite & Postgres allow AVG(DISTINCT), and this is seen a lot in the
tests: apache/datafusion#2408
<apache/datafusion#2408>
—
Reply to this email directly, view it on GitHub
<#148 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AJBQZNIKZ2IJRLYEUKS6R63YIOGQTAVCNFSM6AAAAAAT3NBYQSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNBXHEYDSNZWHA>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
This comment was marked as off-topic.
This comment was marked as off-topic.
And value-wise query results #108 (column-per-line instead of line-per-row). |
We can add a SQLite backend to sqllogictest-bin and see how compatible it is with the original SQLite's sqllogictest. Though we didn't consider compatibility when we started this project, it's never too late to see how compatible we are for now. 🤣
The text was updated successfully, but these errors were encountered: