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

Render queries as Text #133

Merged
merged 8 commits into from
Sep 24, 2019
Merged

Render queries as Text #133

merged 8 commits into from
Sep 24, 2019

Conversation

parsonsmatt
Copy link
Collaborator

No description provided.

@bitemyapp
Copy link
Owner

I've needed this semi-frequently. Let me know if the build needs cleaned up. @parsonsmatt

@Vlix
Copy link
Contributor

Vlix commented Sep 19, 2019

Seems like the same issue I had when trying to make it work locally vs. Travis?
Since I had to up the PSQL version to 10 (for specific JSONB operators), the PSQL port is now 5433, instead of 5432. (or the other way around?) but somewhere that might have to be adjusted before it'll build on Travis.

When working on the JSONB PR locally it would just work with 5432 (or 5433, can't remember), but then Travis complained if it wasn't set to the other before pushing.

From the failed Travis build:

$ echo "CREATE USER esqutest WITH PASSWORD 'esqutest';" | psql postgres
psql: could not connect to server: No such file or directory
	Is the server running locally and accepting
	connections on Unix domain socket "/var/run/postgresql/.s.PGSQL.5433"?
The command "echo "CREATE USER esqutest WITH PASSWORD 'esqutest';" | psql postgres" failed and exited with 2 during .

@bitemyapp
Copy link
Owner

I believe this fixes #6

@bitemyapp bitemyapp merged commit b4bfe53 into master Sep 24, 2019
@bitemyapp bitemyapp deleted the matt/render-query branch September 24, 2019 15:12
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.

3 participants