-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Port other sql_integration test cases to sqllogictest #4870
Labels
enhancement
New feature or request
Comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Is your feature request related to a problem or challenge? Please describe what you are trying to do.
As @jackwener suggested in #4826 (comment), we can port the other part of
sql_integration
tosqllogictest
now.The
sql_integration
tests are indatafusion/core/tests/sql
:The difference:
sql_integration
will ensure the correctness of logical and physical plan.sqllogictest
will ensure the result is correct.For the test cases, if we want to ensure the correctness of the plan, the test should be reserved in
sql_integration
.Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.
The text was updated successfully, but these errors were encountered: