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

Add error symbol/code to sql output when logging output to database #22

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

KyleLilly
Copy link

This PR just adds an additional column code to the error table and base_error view. The code is the same as the error symbol generated by enabling verbose output (C, S, e, t). Having the additional column allowed us to easily filter out the queries that were crashing the server.

I wasn't really sure how to handle the database migration so I went ahead and created a v1.2 to v1.3 script but if you had something else in mind or want me to change column names just let me know.

@srajan3012
Copy link

Hi @KyleLilly
Can you please help me with how you enabled logging to postgres db?
I am new to postgres, I tried --log-to="postgresql://localhost:5432" but it didnt work for me.

Do i need to setup some tables in the postgres before running sqlsmith?

Thanks in advance.

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.

None yet

2 participants