Skip to content

Fixes a Bug in sqlancer #1205

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Eklavya-Sahani
Copy link

fixes a bug in sqlancer that will always throw AssertionError if --log-each-select is set to false. The issue was identified in main.java and has been resolved by updating the logic and improving error handling. The fix has been tested to ensure correct execution and prevent regressions. This improves sqlancer’s stability and accuracy for database testing. 🚀

@Eklavya-Sahani
Copy link
Author

Hi @mrigger @albertZhangTJ , @suyZhong could you please review this PR? It resolves issue #1205 by preventing AssertionError when --log-each-select is false. Thank you!

@Eklavya-Sahani
Copy link
Author

Hi maintainers, the GitHub workflows for this PR haven’t started. Could you please enable workflows for forked contributions so I can get test results and reviews? Thanks!

@Eklavya-Sahani
Copy link
Author

Hi @mrigger, @albertZhangTJ, @suyZhong 👋

This PR fixes issue #1205 by preventing an AssertionError that would occur when --log-each-select is set to false. The logic in Main.java has been updated and tested to ensure proper behavior and avoid regressions.

Also, I noticed that the GitHub Actions workflows haven’t started for this forked PR. Could you please enable workflows for external contributions so the required tests can run? This will help in getting the review and merge process moving forward.

Thank you for your time and help!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant